Felix
  1. Felix
  2. FELIX-3915

[DS] Timing hole between ComponentHolder initial config and registration

    Details

      Description

      Right now when we create a component holder we first try to configure it with any existing configurations and then register it to obtain configuration events for it's pid. Thus there is a timing hole after the query for existing configurations and before the registration when configuration events will be dropped.

      I'm doing the initial configuration after adding the holder to the by-pid map. Instead of missing the configuration entirely, it will be possible that the component would get configured twice. I don't think this can cause any harm.

        Activity

        David Jencks created issue -
        Hide
        David Jencks added a comment -

        Fixed in rev 1449277.

        Show
        David Jencks added a comment - Fixed in rev 1449277.
        David Jencks made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        David Jencks added a comment -

        I thought of a way to avoid possible double configuration on startup, rev 1449401.

        Show
        David Jencks added a comment - I thought of a way to avoid possible double configuration on startup, rev 1449401.
        Hide
        David Jencks added a comment -

        Previous commit still has a timing hole, although it's small. Patch has a way to use the R5 getChangeCount to eliminate the hole, and uses pretty much the same idea as r 1449401 for R4. I can't find a R5 compendium in maven right now, so I'm using reflection. Hoping to improve things...

        Show
        David Jencks added a comment - Previous commit still has a timing hole, although it's small. Patch has a way to use the R5 getChangeCount to eliminate the hole, and uses pretty much the same idea as r 1449401 for R4. I can't find a R5 compendium in maven right now, so I'm using reflection. Hoping to improve things...
        David Jencks made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        14m 3s 1 David Jencks 23/Feb/13 07:28
        Resolved Resolved Closed Closed
        234d 15h 57m 1 David Jencks 16/Oct/13 00:26

          People

          • Assignee:
            David Jencks
            Reporter:
            David Jencks
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development