Houston Putman created SOLR-18072:
-------------------------------------

             Summary: Refactor CollectionApiCommands to be able to easily pass 
in more context
                 Key: SOLR-18072
                 URL: https://issues.apache.org/jira/browse/SOLR-18072
             Project: Solr
          Issue Type: Improvement
            Reporter: Houston Putman


Currently the method signature for these classes is:

{{public void call(ClusterState state, ZkNodeProps message, NamedList<Object> 
results)}}

There's not a lot of additional information that can be passed through. It 
would be great to eventually have these calls be typed instead of having 
generic ZkNodeProps messages (and when we move to exclusively use the v2 API 
this should be pretty easy to do). But in the meantime, trying to add 
information being sent to these commands is very hard because every single 
command needs to be updated.

Instead if we pass in an {{AdminCmdContext}} that contains information relevant 
to the command request, then we can easily expand it in the future without 
having to refactor everything.

{{public void call(AdminCmdContext context, ZkNodeProps message, 
NamedList<Object> results)}}

In the first pass, the context would include things like {{operation}} and 
{{asyncId}} that are currently included in the ZkNodeProps, but are generic so 
can be formalized in the context class. It will also include the ClusterState, 
since that is what it is replacing in the method signature.




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to