hqx871 commented on PR #13303: URL: https://github.com/apache/druid/pull/13303#issuecomment-1378206134
> Done an initial review. @hqx871 , thanks for the changes. Could you please do the following? > > * add an integration test that verifies hadoop ingestion with range partitioning > * add comments explaining the usage and benefits of the new `sample` parameter in the tuning config. I have not add a integration test before, where can I find an example and learn from it? -- 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]
