ACE
  1. ACE
  2. ACE-629

updateOldResourcesStatus has performance issue

        Details

        • Type: Bug Bug
        • Status: Ready for QA Ready for QA (View Workflow)
        • Priority: Normal Normal
        • Resolution: Unresolved
        • Affects Version/s: None
        • Fix Version/s: None
        • Component/s: FN
        • Labels:
          None Labels
        • Customer:
          OMAN-Tel

          Description

          updateOldResourcesStatus has performance issue

          4s in new 1.5s in old

          <soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:ser="http://server.api.nep.mobinets.com/">
          <soapenv:Header/>
          <soapenv:Body>
          <ser:updateOldResourcesStatus>
          <!-Optional:->
          <subscriberNo>25796019</subscriberNo>
          <!-Optional:->
          <status>spare</status>
          </ser:updateOldResourcesStatus>
          </soapenv:Body>
          </soapenv:Envelope>

            Activity

            Descending order - Click to sort in ascending order
            Subversion JIRA

            Links Hierarchy

            		  Documentation

            Invalid license: EXPIRED

            Transition Time In Source Status Execution Times Last Executer Last Execution Date
            Open Bug Open Bug Dev Scheduled Dev Scheduled
            		1m 4s 1 Zein Trabelsi 03/Mar/26 12:44 PM
            Dev Scheduled Dev Scheduled Implementation in progress Implementation in progress
            		3s 1 Zein Trabelsi 03/Mar/26 12:44 PM
            Implementation in progress Implementation in progress To Be Released To Be Released
            		43s 1 Zein Trabelsi 03/Mar/26 12:45 PM
            To Be Released To Be Released Ready for QA Ready for QA
            		23h 23m 1 Khaled Khalil 04/Mar/26 12:08 PM
            Hide
            Permalink
            Zein Trabelsi added a comment -

            it takes now 1s :
            <soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:ser="http://server.api.nep.mobinets.com/">
            <soapenv:Header/>
            <soapenv:Body>
            <ser:updateOldResourcesStatus>
            <!-Optional:->
            <subscriberNo>25796019</subscriberNo>
            <!-Optional:->
            <status>Working pair</status>
            </ser:updateOldResourcesStatus>
            </soapenv:Body>
            </soapenv:Envelope>

            Show
            Zein Trabelsi added a comment - it takes now 1s : <soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:ser="http://server.api.nep.mobinets.com/"> <soapenv:Header/> <soapenv:Body> <ser:updateOldResourcesStatus> <!- Optional: -> <subscriberNo>25796019</subscriberNo> <!- Optional: -> <status>Working pair</status> </ser:updateOldResourcesStatus> </soapenv:Body> </soapenv:Envelope>
            Khaled Khalil made changes -
            Status To Be Released [ 10400 ] Ready for QA [ 10023 ]
            Assignee Zein Trabelsi [ ztrabelsi ] Mbaye Lo [ mbayel ]
            Zein Trabelsi made changes -
            Status Implementation in progress [ 10016 ] To Be Released [ 10400 ]
            Hide
            Permalink
            Zein Trabelsi added a comment -

            changed the logic of finding ospconnection related to fn_service_trail_hop (FnInternalService.getOspConnection (TransferredOspConnection)) .
            we have change the logic of this method (create two new methods in same class with same name and different params) .
            instead of for each hop retrieve node1+node2 + retrieving all ospconnections by node1id and node2id then for each connection get the transferred object and check if port1key = port2key (we may have hundreds connections between two nodes) ==> fetching the related node1 + node2 (By nodeName and nodeType) and port1 + port2 (By NeId and portKey) then fetch directly the related ospconnection (only one match)

            Show
            Zein Trabelsi added a comment - changed the logic of finding ospconnection related to fn_service_trail_hop (FnInternalService.getOspConnection (TransferredOspConnection)) . we have change the logic of this method (create two new methods in same class with same name and different params) . instead of for each hop retrieve node1+node2 + retrieving all ospconnections by node1id and node2id then for each connection get the transferred object and check if port1key = port2key (we may have hundreds connections between two nodes) ==> fetching the related node1 + node2 (By nodeName and nodeType) and port1 + port2 (By NeId and portKey) then fetch directly the related ospconnection (only one match)
            Zein Trabelsi made changes -
            Status Dev Scheduled [ 10014 ] Implementation in progress [ 10016 ]
            Zein Trabelsi made changes -
            Status Open Bug [ 10108 ] Dev Scheduled [ 10014 ]
            Zein Trabelsi made changes -
            Assignee Hassan Abdine [ habdine ] Zein Trabelsi [ ztrabelsi ]
            Mbaye Lo made changes -
            Field Original Value New Value
            Gantt Options Milestone (set to milestone: having a due date but zero effort)
            Planned Start 2026-03-03 24:00 (milestone: set planned start date to due date)
            Planned End 2026-03-03 24:00 (milestone: set planned end date to due date)
            Cancel

              People

              • Assignee:
                Mbaye Lo
                Reporter:
                Mbaye Lo
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Planned Start:
                  Planned End:

                  Drag and Drop