[ 
https://issues.apache.org/jira/browse/SOLR-18113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18057669#comment-18057669
 ] 

Eric Pugh commented on SOLR-18113:
----------------------------------

If nobody is using Raw, not sure we should have it!  YAGNI and all.   I'm 
onboard with the rest of your comments..  Did you get a chance to look at the 
PR?

> Review ZookeeperInfoHandler
> ---------------------------
>
>                 Key: SOLR-18113
>                 URL: https://issues.apache.org/jira/browse/SOLR-18113
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Eric Pugh
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> We should look at a few things in ZookeeperInfoHandler:
>  * ZookeeperInfoHandler currently uses the "raw" request handler, but it 
> returns standard JSON.   If we convert it to json, we don't need "raw" 
> defined in ResponseWriterRegistery anymore.
>  * it looks for a /clusterstate.json from the admin ui, we should update the 
> admin ui to consume the standard format and elminate this werid use case.  
> It's brittle.
>  * contentstream is deprecated in favour of contentwriter.
>  * probably for a v2 version, but think about /admin/zookeeper being 
> /admin/clusterstate and dropping the zookeeper term?



--
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