@diosmosis opened this Issue on August 24th 2015 Member

Currently, it is not possible to store 3rd party reports in archive tables (at least w/o some large hacks) due to this one optimization: https://github.com/piwik/piwik/blob/master/core/ArchiveProcessor/Loader.php#L121

If there are no visits for a period, the Archiver code for a plugin will not be launched. For plugins that only provide 3rd party data, there will never be visits in the log_visit table, so the archiver will never be launched.

To fix this for our LTS version, we can add an event that specifies idSites for whom this optimization should be skipped. This will allow many more plugins to be created for 2.15, while we work on 3.0.

Potential names for the new event include:

  • Archiver.getIdSitesNotUsingLogTables
  • Archiver.getIdSitesDisplayingThirdPartyData
  • Archver.getIdSitesDisplayingNonPiwikData
@tsteur commented on August 26th 2015 Member

I thought quite for a while for a good name that actually describes what we're looking for but couldn't find any. For example one could display third party data but still reuse log_* tables etc eg when importing data from a 3rd party analytics system. After a while I thought how would someone import 3rd party data anyway? Why should they have to use the archiver? The archiver is pretty much there just for the log_* tables. At least currently it's not really built for something else and triggering such an event can be only a workaround (and I'm ok with that if it is needed for another plugin). As it's rather a workaround I don't mind which name we'll pick as none is really good. I'd probably prefer Archiver.getIdSitesNotUsingLogTables though as this is exactly the problem why they have to listen to this event. It's a bit technical but kinda accurate.

The ideal solution would be to either to refactor the archiver (which is planned for 3.0) and/or to offer a kinda new API for them, which could be even just a different method in the archiver for simplicity. For example next to aggregateDayReport() there would be maybe an additional method importDayReport(). We would always call this method even if there are no visits. Beside aggregateMultipleReports() there could be also an importMultipleReports(). It might be too confusing though and having another Import base class might be better. For example an Import base class would not need a method getLogAggregator() etc. The Import class name could be misleading and there would be potentially a better name for it.

I'm not sure how much work it is to introduce such new API but imagine it rather trivial, not sure though. When we detect a plugin defining an importer one would also maybe not have to listen to the CronArchiver.getIdSitesNotUsingTracker anymore since we would kinda know it is not using the tracker but importing data. The MultiSites plugin would still listen to that event since it's using the archiver.

@diosmosis commented on August 26th 2015 Member

This issue is just for enabling developers to display 3rd party data w/ as small a change as possible. 3.0 won't be out for 8 months or so, and I don't think plugin developers (ie, pro devs & their clients) should have to wait for that time to use this functionality. A new 'component' class might be the proper solution, but it doesn't make sense to write all that code when it may be removed for 3.0 and when a single pre-deprecated event will solve it too.

Regarding the name, since it will be removed eventually, I don't really have a preference. If you prefer technically accurate naming, maybe Archiver.getIdSitesWithNoVisitsToForceArchivingFor or Archiver.getIdSitesToForceArchivingForEvenIfNoVisits would do?

@tsteur commented on August 27th 2015 Member

I'm fine with any event name as long as it doesn't contain force since it leads to wrong expectations IMO. Eg because there can be still reasons why it would not be archived...

I'm fine with using the event as a workaround and to develop maybe a new base class in Piwik 3.0 for importing data.

This Issue was closed on September 22nd 2015
Powered by GitHub Issue Mirror