[
https://issues.apache.org/jira/browse/SOLR-13331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16845806#comment-16845806
]
Jason Gerlowski commented on SOLR-13331:
----------------------------------------
bq. We need to ensure that tests run with json+ javabin too.
I'm totally behind any effort to better randomize the wire format used by our
tests. David's SOLR-11872 would be a huge step forward in that regard. I'm
for that effort independent of this issue here.
That said, I don't think better testing really fixes the issue here if the
underlying abstraction remains broken. We're addressing the "symptoms"
(serialization issues with individual APIs/features), but ignoring the
"disease" (the broken BAUCS abstraction). Without fixing that, we're going to
see these issues crop up for years as new features get added.
> 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]
