Details
-
Type:
Improvement
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 4.0-ALPHA
-
Component/s: None
-
Labels:None
Description
There are legitimate use-cases for folks to extend QuerySenderListener. For example, at Etsy, we would like toggle warm-up queries according to feature flags in our business logic. Unfortunately, this class is declared with the default access level...
Activity
- All
- Comments
- Work Log
- History
- Activity
- Transitions
Yeah, i'm not sure that there is any reason why QuerySenderListener and AbstractSolrEventListener were never made public.
i don't see that there is much extinsible value add beyond the "if (condition) super.method(...)" style delegation, but as you mention: even that has it's advantages.
if you have other suggestiosn for refactoring the class to be more useful when extending, please feel free to open other issues (and submit patches)
Committed revision 1095545.