abhishekrb19 opened a new pull request, #16174: URL: https://github.com/apache/druid/pull/16174
#### Issue: The `markUsed` API accepts either an `interval` or `segmentIds`. When using segment IDs, the underlying query uses an `IN` clause with the user-provided values inlined directly in the query instead of being parameterized. #### Fix: This patch fixes that by adding the following utilities specifically for queries that contain an `IN` clause: - `getParameterizedInConditionForColumn()` - `bindColumnValuesToQueryWithInCondition()` #### Other related changes: Renamed the following functions for consistency and code hygiene: - `appendConditionForIntervalsAndMatchMode()` to `getConditionForIntervalsAndMatchMode()`. Pass only select arguments to the function. - `bindQueryIntervals()` to `bindIntervalsToQuery()` This PR has: - [x] been self-reviewed. - [ ] a release note entry in the PR description. - [x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links. - [x] been tested in a test Druid cluster. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
