bugCERTI - Bugs: bug #51133, Registering Objects Inefficient

 
 

bug #51133: Registering Objects Inefficient

Submitter:  Joris Goosen <jorisgoosen>
Submitted:  Mon 29 May 2017 07:58:55 AM UTC
   
 
Category:  CERTI Severity:  * 3 - Normal
Item Group:  Action request Status:  Fixed
Privacy:  Public Assigned to:  cvannier
Open/Closed:  Open Release:  * GIT-master
Reproducibility:  Every Time Fixed Release:  * NextRelease
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 18 Dec 2017 04:20:11 PM UTC, comment #3: 

Dear Joris,

Your patch has been applied on branch dev-scalian/cvannier

Our current test suite did not raise any issue.

You can fetch the aforementioned branch from commit #60b060

Please come back to us if you encounter any issue.

Thank you again for your patch.

Best regards,
Clément

Clément Vannier <cvannier>
Group Member
Mon 17 Jul 2017 02:10:16 PM UTC, comment #2: 

Dear Joris,

Thanks for your investigation and for providing a patch.
I am sorry for the late feedback, I haven't seen your request.

I will have a look into your patch and, maybe, this will be integrated in the next release if this doesn't break some others tests.

Many thanks,
Best regards,
JB Ch

JbCh <jb_ch>
Group administrator
Mon 29 May 2017 08:02:25 AM UTC, comment #1: 

The registering is slow because there was a loop over all objects to check for conflicting names. See patch for a fix, in my case a registering time of roughly 420s went down to 15s.

Joris Goosen <jorisgoosen>
Mon 29 May 2017 07:58:55 AM UTC, original submission:  

Registering a lot of objects on the RTI slows down (way) more than linearly..

Joris Goosen <jorisgoosen>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #40807:  FixSlowObjectRegistering.patch added by jorisgoosen (8KiB - application/octet-stream - Patch fixes the objectset)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by cvannier (Posted a comment)
  • -email is unavailable- added by jb_ch (Posted a comment)
  • -email is unavailable- added by jorisgoosen (Submitted the item)
  •  

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-12-18 cvannier Item GroupNone Action request
        StatusNone Fixed
        Assigned toNone cvannier
        ReproducibilityNone Every Time
        Fixed ReleaseGIT-master NextRelease
    2017-05-29 jorisgoosen Attached File- Added FixSlowObjectRegistering.patch, #40807

    Back to the top

    Powered by Savane 3.13-cf05.
    Corresponding source code