paul-rogers commented on PR #13774: URL: https://github.com/apache/druid/pull/13774#issuecomment-1441107999
To help move this along, can we split the PR into parts? Maybe one or two tests per PR? To do that, create a new PR with one or two tests. Create another with another one or two. These can be done in parallel if there are no common files (the ideal case), or sequentially if there are shared files. Once all but two of the tests have been done in their own PR, merge master (with those tests) into this branch, leaving the last couple of tests. In particular, the security-related tests should be their own PR since they are quite complex by themselves. For extra credit, revise the `docker-compose.yaml` files to us the new templating mechanism. If that is too much of an ask, go ahead and do a PR as-is, and I'll convert the files, since I can do so quickly (having written the generator code...) -- 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: commits-unsubscr...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
