[GitHub] [solr] epugh commented on pull request #1348: Adding support for compression within the ZkCLI tool

["epugh (via GitHub)" <gi...@apache.org>]

epugh commented on PR #1348:
URL: https://github.com/apache/solr/pull/1348#issuecomment-1426096269

   This looks like it does the job.  However, I was sort of thinking that compression would be more of a wrapper around the underlying data structure?   Kind of like a `ZipInputStream` might wrap a `InputStream`?    It seems like there is a lot of logic in the CLI around all the compression, and that it might be ripe for bugs...    
   
   Maybe thinking it would look more like `ZKReader reader = getZKReader()` and you woudn't see all the compression logic?
   
   Someday I want to break up ZKCli.java and make it easier to reason about.   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org