Github user moshebla commented on a diff in the pull request:
https://github.com/apache/lucene-solr/pull/385#discussion_r192651989
--- Diff:
solr/core/src/java/org/apache/solr/update/DirectUpdateHandler2.java ---
@@ -952,12 +941,15 @@ public void split(SplitIndexCommand cmd) throws
IOException {
*
* @param cmd - cmd apply to IndexWriter
* @param writer - IndexWriter to use
- * @param updateTerm - used if this cmd results in calling {@link
IndexWriter#updateDocument}
*/
- private void updateDocOrDocValues(AddUpdateCommand cmd, IndexWriter
writer, Term updateTerm) throws IOException {
+ private Term updateDocOrDocValues(AddUpdateCommand cmd, IndexWriter
writer) throws IOException {
assert null != cmd;
final SchemaField uniqueKeyField =
cmd.req.getSchema().getUniqueKeyField();
final String uniqueKeyFieldName = null == uniqueKeyField ? null :
uniqueKeyField.getName();
+ List<SolrInputDocument> docs = cmd.computeFinalFlattenedSolrDocs();
--- End diff --
If i pass isBlock as false in case of an inPlaceUpdate to getIdTerm, the
overwrite test seems to fail, as one of the documents is not overwritten.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
