[#SFOS-1150] Have AbstractTargetedCondition attempt to resolve the target on demand

SmartFrog

Have AbstractTargetedCondition attempt to resolve the target on demand

Created: 19/Mar/09 04:23 PM (GMT) Updated: 20/Mar/09 04:40 PM (GMT)

Return to search

Component/s:

.sfCore

Affects Version/s:

3.17.010

Fix Version/s:

3.17.010

Time Tracking:

Not Specified

Compatibility:

may break builds or test process

Description

« Hide

Currently the IsAlive() test and the like require the CD to be deployed before they can begin polling it, because they try and resolve the reference on startup.

If instead the resolution was postponed to the condition, the polling operation in a WaitFor operation will even handle slow starting components .

All

Comments

Work Log

Change History

Sort Order:

Steve Loughran made changes - 19/Mar/09 04:23 PM (GMT)

Field	Original Value	New Value
Assignee	Julio Guijarro [ julgui ]	Steve Loughran [ stevel ]

Steve Loughran made changes - 19/Mar/09 04:23 PM (GMT)

Status

Open [ 1 ]

In Progress [ 3 ]

Steve Loughran made changes - 19/Mar/09 05:12 PM (GMT)

Summary	Have AbstractTargetedCondition attempt to resolve the target every time it is started, so that IsAlive and the like will even work for targets that are not yet deployed	Have AbstractTargetedCondition attempt to resolve the target on demand
Description	Currently the IsAlive() test and the like require the CD to be deployed before they can begin polling it, because they try and resolve the reference on startup.# If instead the resolution was postponed to the condition, the polling operation in a WaitFor operation will even handle slow starting components .	Currently the IsAlive() test and the like require the CD to be deployed before they can begin polling it, because they try and resolve the reference on startup. If instead the resolution was postponed to the condition, the polling operation in a WaitFor operation will even handle slow starting components .

Steve Loughran made changes - 19/Mar/09 05:13 PM (GMT)

Status

In Progress [ 3 ]

Open [ 1 ]

[ Permlink | « Hide ]

Steve Loughran added a comment - 19/Mar/09 05:14 PM (GMT)

I've checked this change in, it all works. But tests may fail if they expect a failure to resolve to fail differently. Perhaps I should add a condition resolveTargetOnStartup, which does a check there for backwards compatibiltiy.

[ Permlink | « Hide ]

Steve Loughran added a comment - 20/Mar/09 04:40 PM (GMT)

done; the hadoop components have picked it up. Resolution is now done early unless declared otherwise; the hadoop checkers are all late binding

Steve Loughran made changes - 20/Mar/09 04:40 PM (GMT)

Status	Open [ 1 ]	Resolved [ 5 ]
Fix Version/s		3.17.x [ 10160 ]
Resolution		Fixed [ 1 ]