jetty.project icon indicating copy to clipboard operation
jetty.project copied to clipboard

Review Container.InheritedListener

Open joakime opened this issue 7 years ago • 6 comments

Based on a prelim review, it seems this is really 2 different interfaces.

joakime avatar Mar 19 '18 22:03 joakime

We perhaps need a way to propagate a bean to all container components on the tree (which is what InheritedListener does), but we also need a way to have a leaf component tell its parent (and recursively to its ancestors) what beans it's interested in.

This would allow to add a bean to HttpClient but that bean is actually used by the ManagedSelectors that are deeper nodes of HttpClient, many layers below.

Also, we should make these listeners implement java.util.EventListener where is the case.

sbordet avatar Mar 19 '18 22:03 sbordet

This issue has been automatically marked as stale because it has been a full year without activit. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Nov 20 '19 14:11 stale[bot]

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Nov 21 '20 07:11 stale[bot]

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar May 03 '22 00:05 github-actions[bot]

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar May 06 '23 00:05 github-actions[bot]

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar May 07 '24 00:05 github-actions[bot]