Github user moshebla commented on a diff in the pull request:
https://github.com/apache/lucene-solr/pull/385#discussion_r192638234
--- 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 --
Yeah, but that means we wouldn't have a way to figure out if the update is
a block, which is a parameter which needs to be passed to getIdTerm.
I'm not as familiar with the internals, would it always be false during an
inPlace update?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
