Github user dsmiley commented on the issue:

    https://github.com/apache/lucene-solr/pull/528
  
    I like that the zk version of processCommit completely overrides the 
superclass since the logic is much more complicated.  But it's a little 
unfortunate to call super.processCommit which will have some redundant 
setupRequest.  One slight change would be for these method calls to change back 
to call localCommit() as they did before, but also put the cmd.setVersion logic 
at the front of localCommit's logic since this is always done first.


---

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

Reply via email to