CNDB-16021: CNDB-15280: Remove user data from AbstractReadQuery.toCQLString

[michaelsembwever]

https://github.com/riptano/cndb/issues/16021

Port into main-5.0 commit 4517338

Will be merged to main-5.0 after mck-cndb-15760-main-5.0 is merged.

CNDB-16021: CNDB-15280: Remove user data from AbstractReadQuery.toCQLString

Replace column values by '?' when converting internal read queries to CQL's SELECT queries, so user data don't end up in logs or any other unprotected place.

CQL queries printed in tracing and user-facing errors still expose the unredacted data.

[github-actions]

Checklist before you submit for review

• This PR adheres to the Definition of Done
• 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
• All new files should contain the DataStax copyright header instead of the Apache License one

[adelapena]

Overall looks good, but it seems the new SlowQueryLoggerTest is failing.

[adelapena]

LGTM, +1.

[michaelsembwever]

failed tests not reproduced locally. ninja fix squashed in for check failure on SAICodecUtils

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
87.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[cassci-bot]

❌ Build ds-cassandra-pr-gate/PR-2149 rejected by Butler

---

8 regressions found
See build details here

---

Found 8 new test failures

Test	Explanation	Runs	Upstream
o.a.c.cql3.validation.operations.AggregationQueriesTest.testAggregationQueryShouldNotTimeoutWhenItExceedesReadTimeout (compression)	NEW	🔴🔴	2 / 25
o.a.c.distributed.test.DropUDTWithRestartTest.loadCommitLogAndSSTablesWithDroppedColumnTestCC50	NEW	🔴🔴	22 / 25
o.a.c.distributed.test.DropUDTWithRestartTest.loadCommitLogAndSSTablesWithDroppedColumnTestCassandra41	NEW	🔴🔴	1 / 25
o.a.c.distributed.test.DropUDTWithRestartTest.loadCommitLogAndSSTablesWithDroppedColumnTestCassandra5	NEW	🔴🔴	1 / 25
o.a.c.index.SecondaryIndexManagerTest.testIndexRebuildWhenAddingSStableViaRemoteReload (compression)	NEW	🔴🔴	20 / 25
o.a.c.metrics.TrieMemtableMetricsTest.testContentionMetrics (compression)	NEW	🔵🔴	9 / 25
o.a.c.nodes.NodesPersistenceTest.testUpdateTokens (compression)	NEW	🔵🔴	0 / 25
o.a.c.utils.binlog.BinLogTest.testTruncationReleasesLogSpace (compression)	NEW	🔴🔵	1 / 25

---

No known test failures found