cheddar commented on PR #14466: URL: https://github.com/apache/druid/pull/14466#issuecomment-1603905986
The other_modules tests are complaining because the SqlTaskResource apparently never had tests put on it, so there's no coverage. This PR is just fixing the known compatibility issue and not fixing the fact that tests were never actually created to help catch such issues in the first place. There also appears to be a flakey test in processing around HttpConfig. I've seen it pass and fail multiple times over now, I do not know why, but the CI has been pretty flakey with this PR and fighting with it to get clean green checks is just going to waste energy. So I'm going to merge as is. -- 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]
