[GitHub] [solr] dsmiley commented on pull request #1504: SOLR-7609 ShardSplitTest NPE investigation

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

dsmiley commented on PR #1504:
URL: https://github.com/apache/solr/pull/1504#issuecomment-1495363648

   I'm glad you are digging into this test failure!  You've left lots of breadcrumbs for me to follow.
   I like adding more checks to insist there is a version if we're forwarded an update from a leader.  If some code paths don't verify it, it was likely forgotten.  It's not clear to me if you've root caused why it can be null.
   
   There may be a relation to #1484 which I have yet to merge, as I'm trying to be real thorough first.  I thought maybe the change there might fix this one but no it doesn't.
   
   > there is a delete that is reported successful, but not applied
   
   Clearly a problem.
   
   


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