Skip to content

Conversation

@pkolaczk
Copy link

When a wildcard selection is used, the score column
must be included in the selection.

Fixes:
NoHostAvailable: ('Unable to complete the operation against any hosts', {<Host: 10.87.217.86:9042 us-central1>: <Error from server: code=0000 [Server error] message="java.lang.ClassCastException:
class org.apache.cassandra.serializers.UTF8Serializer cannot be cast to class org.apache.cassandra.db.marshal.VectorType$VectorSerializer
(org.apache.cassandra.serializers.UTF8Serializer and org.apache.cassandra.db.marshal.VectorType$VectorSerializer are in unnamed module of loader org.apache.felix.framework.BundleWiringImpl$BundleClassLoader @446c3920)">})

@github-actions
Copy link

github-actions bot commented Mar 17, 2025

Checklist before you submit for review

  • Make sure there is a PR in the CNDB project updating the Converged Cassandra version
  • Use NoSpamLogger for log lines that may appear frequently in the logs
  • Verify test results on Butler
  • Test coverage for new/modified code is > 80%
  • Proper code formatting
  • Proper title for each commit staring with the project-issue number, like CNDB-1234
  • Each commit has a meaningful description
  • Each commit is not very long and contains related changes
  • Renames, moves and reformatting are in distinct commits

@pkolaczk pkolaczk requested a review from adelapena March 17, 2025 10:35
Copy link

@adelapena adelapena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The patch looks good, but I think the same error can still be reproduced with GROUP BY:

createTable("CREATE TABLE %s (k int, c int, v text, PRIMARY KEY(k, c))");
analyzeIndex();
var result = execute("SELECT * FROM %s GROUP BY k, c ORDER BY v BM25 OF 'apple' LIMIT 3");
assertThat(result).hasSize(1);

@pkolaczk pkolaczk force-pushed the c13330-select-star-order-by branch 3 times, most recently from 741f49c to 1d72089 Compare March 17, 2025 13:16
Copy link

@adelapena adelapena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, let's wait for the new CI run.

Copy link

@adelapena adelapena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last changes look good to me, let's see if CI looks good now.

@pkolaczk
Copy link
Author

testANNOptionsWithMixedDS10AndDS11 works for me locally, probably a flaky.

When a wildcard selection is used, the score column
must be included in the selection.

Fixes:
NoHostAvailable: ('Unable to complete the operation against any hosts', {<Host: 10.87.217.86:9042 us-central1>: <Error from server: code=0000 [Server error] message="java.lang.ClassCastException:
class org.apache.cassandra.serializers.UTF8Serializer cannot be cast to class org.apache.cassandra.db.marshal.VectorType$VectorSerializer
(org.apache.cassandra.serializers.UTF8Serializer and org.apache.cassandra.db.marshal.VectorType$VectorSerializer are in unnamed module of loader org.apache.felix.framework.BundleWiringImpl$BundleClassLoader @446c3920)">})
@pkolaczk pkolaczk force-pushed the c13330-select-star-order-by branch from 2a8296c to 2a21a7a Compare March 18, 2025 10:26
@sonarqubecloud
Copy link

@cassci-bot
Copy link

✔️ Build ds-cassandra-pr-gate/PR-1636 approved by Butler


Approved by Butler
See build details here

@pkolaczk pkolaczk merged commit 88675ba into main Mar 18, 2025
474 of 478 checks passed
@pkolaczk pkolaczk deleted the c13330-select-star-order-by branch March 18, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants