? ? Success

User tests: Successful: Unsuccessful:

avatar chrisdavenport
chrisdavenport
1 Nov 2016

Summary of Changes

Merely adds the peak memory usage to the output of the Smart Search CLI indexer.

Testing Instructions

Run the CLI indexer and look for the new line in the output. Here's an example:

Starting Indexer
Setting up Smart Search plugins
Setup 8 items in 0.154 seconds.
 * Processed batch 1 in 0.052 seconds.
Total Processing Time: 0.207 seconds.
Peak memory usage: 6,291,456 bytes

Documentation Changes Required

None.

avatar chrisdavenport chrisdavenport - open - 1 Nov 2016
avatar chrisdavenport chrisdavenport - change - 1 Nov 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Nov 2016
Labels Added: ? ?
avatar joomla-cms-bot joomla-cms-bot - change - 1 Nov 2016
Category CLI Language & Strings
avatar alikon
alikon - comment - 1 Nov 2016

i've noticed some differences on how behave the display of the "Peak memory usage" from CLI to WEB

  • from the CLI we have only one display at end of the index process
    new_cli

  • from the WEB we have one display for each batch process
    web_indexer

avatar mbabker
mbabker - comment - 1 Nov 2016

CLI is doing it all in one process. Web is doing it in batches so the memory use is per batch (per process since it's multiple AJAX requests).

avatar alikon
alikon - comment - 1 Nov 2016
avatar mbabker
mbabker - comment - 1 Nov 2016

Right, CLI does run in batches, but it's all in the context of one long running process versus the web doing it in several processes. So unless Chris changes it to track the memory use per batch, just from reading the discussions here I think he's tracking the memory use for the entire process in each context (so one of the AJAX requests for the web and the full CLI run).

avatar alikon
alikon - comment - 1 Nov 2016

tracking the memory use for the entire process

yes i think this too
i was trying to compare both way results to calculate the "gain", maybe i have wanted to compare apples with pears ;)

avatar alikon alikon - test_item - 6 Nov 2016 - Tested successfully
avatar alikon
alikon - comment - 6 Nov 2016

I have tested this item successfully on 2a7af71


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12679.

avatar brianteeman
brianteeman - comment - 6 Dec 2016

I have tested this item successfully on 2a7af71


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12679.

avatar brianteeman brianteeman - test_item - 6 Dec 2016 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 6 Dec 2016
Status Pending Ready to Commit
Labels
avatar jeckodevelopment
jeckodevelopment - comment - 6 Dec 2016

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12679.

avatar jeckodevelopment jeckodevelopment - change - 6 Dec 2016
Milestone Added:
avatar rdeutz rdeutz - close - 6 Dec 2016
avatar rdeutz rdeutz - change - 6 Dec 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-12-06 21:55:52
Closed_By rdeutz
avatar rdeutz rdeutz - close - 6 Dec 2016
avatar rdeutz rdeutz - merge - 6 Dec 2016
avatar rdeutz rdeutz - reference | 846ba67 - 6 Dec 16
avatar rdeutz rdeutz - merge - 6 Dec 2016

Add a Comment

Login with GitHub to post a comment