Re: Refactoring ZkStateReader

2022-12-31 Thread David Smiley
Makes sense -- thanks for tackling this! ~ David Smiley Apache Lucene/Solr Search Developer http://www.linkedin.com/in/davidwsmiley On Fri, Dec 30, 2022 at 5:37 PM Justin Sweeney wrote: > Hi all, > > ZkStateReader ( > > https://github.com/apache/solr/blob/main/solr/solrj-zookeeper/src/java/org

Re: Refactoring ZkStateReader

2022-12-30 Thread Noble Paul
Thanks Justin This is one of the beasts in Solr On Sat, Dec 31, 2022, 9:37 AM Justin Sweeney wrote: > Hi all, > > ZkStateReader ( > > https://github.com/apache/solr/blob/main/solr/solrj-zookeeper/src/java/org/apache/solr/common/cloud/ZkStateReader.java > ) > is a complicated class with thousand

Refactoring ZkStateReader

2022-12-30 Thread Justin Sweeney
Hi all, ZkStateReader ( https://github.com/apache/solr/blob/main/solr/solrj-zookeeper/src/java/org/apache/solr/common/cloud/ZkStateReader.java) is a complicated class with thousands of lines and numerous inner classes. This makes approaching this class overly complex. I was recently investigating