diff options
author | Thomas Steur <tsteur@users.noreply.github.com> | 2018-11-21 01:41:26 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-11-21 01:41:26 +0300 |
commit | d8a4d6dde7b459b6084d6a00af081f5a7e706e99 (patch) | |
tree | 2ed27e26b6c3eba8edcd25a18269624bd8deac8e /plugins/Actions | |
parent | 31ed49c5d8e81f5d68a2cd561c2a51d7cc31712f (diff) |
Flattened "entry page" and "exit page" take a long time to be generated (#13496)
fix https://github.com/matomo-org/matomo/issues/11645
Debugged this on demo.matomo.org....
Wasn't as guessed https://github.com/matomo-org/matomo/blame/3.5.1/plugins/Actions/DataTable/Filter/Actions.php#L65-L67 but the flat parameter. After applying it the performance improved from minutes to less than a second.
Diffstat (limited to 'plugins/Actions')
-rw-r--r-- | plugins/Actions/API.php | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/plugins/Actions/API.php b/plugins/Actions/API.php index 1b3cf888c5..bb4489e295 100644 --- a/plugins/Actions/API.php +++ b/plugins/Actions/API.php @@ -167,11 +167,11 @@ class API extends \Piwik\Plugin\API * Returns a DataTable with analytics information for every unique entry page URL, for * the specified site, period & segment. */ - public function getEntryPageUrls($idSite, $period, $date, $segment = false, $expanded = false, $idSubtable = false) + public function getEntryPageUrls($idSite, $period, $date, $segment = false, $expanded = false, $idSubtable = false, $flat = false) { Piwik::checkUserHasViewAccess($idSite); - $dataTable = $this->getPageUrls($idSite, $period, $date, $segment, $expanded, $idSubtable); + $dataTable = $this->getPageUrls($idSite, $period, $date, $segment, $expanded, $idSubtable, false, $flat); $this->filterNonEntryActions($dataTable); return $dataTable; } @@ -180,11 +180,11 @@ class API extends \Piwik\Plugin\API * Returns a DataTable with analytics information for every unique exit page URL, for * the specified site, period & segment. */ - public function getExitPageUrls($idSite, $period, $date, $segment = false, $expanded = false, $idSubtable = false) + public function getExitPageUrls($idSite, $period, $date, $segment = false, $expanded = false, $idSubtable = false, $flat = false) { Piwik::checkUserHasViewAccess($idSite); - $dataTable = $this->getPageUrls($idSite, $period, $date, $segment, $expanded, $idSubtable); + $dataTable = $this->getPageUrls($idSite, $period, $date, $segment, $expanded, $idSubtable, false, $flat); $this->filterNonExitActions($dataTable); return $dataTable; } @@ -216,11 +216,11 @@ class API extends \Piwik\Plugin\API * for the given site, time period & segment. */ public function getEntryPageTitles($idSite, $period, $date, $segment = false, $expanded = false, - $idSubtable = false) + $idSubtable = false, $flat = false) { Piwik::checkUserHasViewAccess($idSite); - $dataTable = $this->getPageTitles($idSite, $period, $date, $segment, $expanded, $idSubtable); + $dataTable = $this->getPageTitles($idSite, $period, $date, $segment, $expanded, $idSubtable, $flat); $this->filterNonEntryActions($dataTable); return $dataTable; } @@ -230,11 +230,11 @@ class API extends \Piwik\Plugin\API * for the given site, time period & segment. */ public function getExitPageTitles($idSite, $period, $date, $segment = false, $expanded = false, - $idSubtable = false) + $idSubtable = false, $flat = false) { Piwik::checkUserHasViewAccess($idSite); - $dataTable = $this->getPageTitles($idSite, $period, $date, $segment, $expanded, $idSubtable); + $dataTable = $this->getPageTitles($idSite, $period, $date, $segment, $expanded, $idSubtable, $flat); $this->filterNonExitActions($dataTable); return $dataTable; } |