clintropolis commented on pull request #12324: URL: https://github.com/apache/druid/pull/12324#issuecomment-1063388996
>@clintropolis looks like we still have a bunch of failing tests, any reason we merged this before tests passed? ah, the jdk15 tests were flaking out due to a [travis side issue](https://travis-ci.community/t/update-install-jdk-sh-with-the-new-revision/12761), though they seem to have started passing again so maybe I should've re-ran them, my bad. The actual failing tests were coverage related due to code change being in 'processing', while tests were in 'sql', and since this fixed an issue with another change that was just merged I just went ahead and merged. -- 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]
