Github user moshebla commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/501#discussion_r234872041
  
    --- Diff: solr/core/src/java/org/apache/solr/update/AddUpdateCommand.java 
---
    @@ -194,20 +197,33 @@ public String getHashableId() {
           return null; // caller should call getLuceneDocument() instead
         }
     
    -    String rootId = getHashableId();
    -
    -    boolean isVersion = version != 0;
    +    final String rootId = getHashableId();
    +    final boolean hasVersion = version != 0;
    +    final SolrInputField versionSif = hasVersion? 
solrDoc.get(CommonParams.VERSION_FIELD): null;
    --- End diff --
    
    OK I changed this and ran SolrExampleTests which seem to pass.
    I'm currently running the solr-core test suite.


---

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

Reply via email to