cpoerschke commented on code in PR #2103: URL: https://github.com/apache/solr/pull/2103#discussion_r1414316684
########## solr/CHANGES.txt: ########## @@ -136,6 +136,8 @@ Bug Fixes * SOLR-17093: Collection restore API command now returns "requestid" when executed asynchronously like other APIs (Tomás Fernández Löbbe) +* SOLR-16952: In distributed i.e. multi-shard cloud mode returned dense vector 'fl' fields are now a list of numbers instead of strings. (Qais Qadri, Christine Poerschke) Review Comment: My understanding is that this does not impact the search -- in fact one can do a non-vector search and request the vector field in the `fl` -- the search happens on the shards and the merging is based on scores. The type in the response being non-numeric would impact on clients using that part of the response _and_ from re-reading the https://lists.apache.org/thread/xnfrxvm0p8yknrrszcqn0dd0vfg3wtvj user mailing list thread is appears that it _might_ also impact debug/explain functionality, which from vague memory would run on the federating node rather than on the shard nodes. Also tagging @alessandrobenedetti here for awareness and I'll also reply on the mailing list thread. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
