You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@solr.apache.org by "Mikhail Khludnev (Jira)" <ji...@apache.org> on 2023/03/02 20:06:00 UTC
[jira] [Comment Edited] (SOLR-16681) Replacing uniqueKey field via fl doesn't work in distributed since 9.0
[ https://issues.apache.org/jira/browse/SOLR-16681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17695871#comment-17695871 ]
Mikhail Khludnev edited comment on SOLR-16681 at 3/2/23 8:05 PM:
-----------------------------------------------------------------
I think [https://github.com/apache/solr/pull/1384] is ready.
Remaining doubts are:
* is it fine to piggyback on those tests or it's worth to bring up a new one?
* I'm sure there's a better wording in exception message. Which one?
* I feel like it's really hard to explain in documentation especially giving that it's something which doesn't work. I think exception is explicit enough.
I wish put it into 9.2 as is.
was (Author: mkhludnev):
I think like [https://github.com/apache/solr/pull/1384]
Remaining doubts are:
* is it fine to piggyback on those tests or it's worth to bring up a new one?
* I'm sure there's a better wording in exception message. Which one?
* I feel like it's really hard to explain in documentation especially giving that it's something which doesn't work. I think exception is explicit enough.
I wish put it into 9.2 as is.
> Replacing uniqueKey field via fl doesn't work in distributed since 9.0
> ----------------------------------------------------------------------
>
> Key: SOLR-16681
> URL: https://issues.apache.org/jira/browse/SOLR-16681
> Project: Solr
> Issue Type: Bug
> Security Level: Public(Default Security Level. Issues are Public)
> Reporter: Mikhail Khludnev
> Priority: Minor
> Fix For: 9.2
>
> Attachments: image-2023-02-26-09-48-31-974.png, image-2023-02-26-09-48-59-323.png
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> User reported a use case which were working before 9 but not anymore.
> The point is to logically replace a field (hereby it's {{id}} {-}but it might behave same for any other{-})
> {{fl=old_id:id,id:new_id}}
> I'd say it's a kind of {{{}$mv new_id id{}}}. I've made a simple [reproducer|https://github.com/apache/solr/pull/1384].
> it fails with
> {code:java}
> - org.apache.solr.cloud.TestCloudPseudoReturnFields.test_mv_fl (:solr:core)
> [7597|https://github.com/apache/solr/actions/runs/4276380498/jobs/7444431149#step:4:7598] Test output: /tmp/src/solr/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.cloud.TestCloudPseudoReturnFields.txt
> [7598|https://github.com/apache/solr/actions/runs/4276380498/jobs/7444431149#step:4:7599] Reproduce with: gradlew :solr:core:test --tests "org.apache.solr.cloud.TestCloudPseudoReturnFields.test_mv_fl" -Ptests.jvms=96 "-Ptests.jvmargs=-XX:TieredStopAtLevel=1 -XX:+UseParallelGC -XX:ActiveProcessorCount=1 -XX:ReservedCodeCacheSize=120m" -Ptests.seed=4CCEBE424880B511 -Ptests.file.encoding=US-ASCII
> {code}
> . [Then I applied it|https://github.com/mkhludnev/solr/tree/SOLR-16681-proves-work-before] to revision preceding [SOLR-9376|https://github.com/mkhludnev/solr/commit/6ff81312607dd5d33f87dc52aed9d52938dc6883#diff-e72c1360a9b0097bcc01e269a94045ce5af1fdc93ae3021ca7e8d1b26d46bdcb] and test passed. For me it seems like:
> - Solr behaved like described before, but it does not after SOLR-9376.
> - Should we reproduce this behavior or we can suggest a workaround?
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org