[
https://issues.apache.org/jira/browse/SOLR-13331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16845818#comment-16845818
]
Jason Gerlowski commented on SOLR-13331:
----------------------------------------
Also, I haven't forgotten about your PR Thomas. I'm hoping to review it soon.
That said, SOLR-11872 has gained some recent attention, and that might be a
better path forward for expanding our JavaBinCodec test coverage in the long
term. That JIRA is a much larger effort, but would ultimately bring codec
randomization to a large group of tests, since many already use the base class
involved there. Hard to tell whether that removes the need for your proposed
EmbeddedSolrTestBase or not until I find time to take a look. But apologies
again for the delay and I'm hoping to get to it soon.
> Atomic Update Multivalue remove does not work
> ---------------------------------------------
>
> Key: SOLR-13331
> URL: https://issues.apache.org/jira/browse/SOLR-13331
> Project: Solr
> Issue Type: Bug
> Security Level: Public(Default Security Level. Issues are Public)
> Components: UpdateRequestProcessors
> Affects Versions: 7.7, 7.7.1, 8.0
> Environment: Standalone Solr Server
> Reporter: Thomas Wöckinger
> Assignee: Jason Gerlowski
> Priority: Critical
> Labels: patch, pull-request-available, ready-to-commit, test
> Fix For: 7.7.2, 8.1, master (9.0)
>
> Attachments: Fix-SOLR13331-Add-toNativeType-implementations.patch,
> SOLR-13331.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> When using JavaBinCodec the values of collections are of type
> ByteArrayUtf8CharSequence, existing field values are Strings so the remove
> Operation does not have any effect.
> The relevant code is located in class AtomicUpdateDocumentMerger method
> doRemove.
> The method parameter fieldVal contains the collection values of type
> ByteArrayUtf8CharSequence, the variable original contains the collection of
> Strings
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]