sakshichitnis27 opened a new pull request, #20553: URL: https://github.com/apache/druid/pull/20553
Fixes #19036. ### Description The classic batch loader currently enables schema discovery while sampling Druid input sources. Complex metric values returned during reingestion are then treated as dimensions, causing the sampler to fail with `Unhandled type: COMPLEX`. This PR disables schema discovery only for Druid reingestion samples, restoring the behavior from before the Delta sampling change. Other fixed-format sources, including Delta, continue to use schema discovery. Regression tests cover both request shapes. #### Release note Fixed classic batch loader sampling when reingesting from Druid datasources that contain complex metrics. ### Testing - `npx jest --config jest.config.js src/utils/sampler.spec.ts --runInBand` - `npx jest --config jest.config.js --testPathIgnorePatterns=e2e-tests --runInBand` (167 suites, 852 tests) - `npm run typecheck` - `npm run eslint` - `npm run sasslint` - `npm run prettify-check` <hr> ##### Key changed/added classes in this PR - `sampleForConnect` <hr> This PR has: - [x] been self-reviewed. - [x] a release note entry in the PR description. - [x] added unit tests to cover the changed code paths. -- 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]
