You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/12/21 20:23:12 UTC

[GitHub] [druid] suneet-s commented on pull request #12070: Removing unused processing threadpool on broker

suneet-s commented on pull request #12070:
URL: https://github.com/apache/druid/pull/12070#issuecomment-999070182


   > Travis will fail due to lack of coverage on openJDK11 and openJDK15 server module tests. This is expected as there is a piece of code in `BrokerProcessingModule` that invokes `JvmUtils.getRuntimeInfo().getDirectMemorySizeBytes();` and JDK 9 and above do not support checking for direct memory size. The server module tests involving JDK8 passes on Travis
   
   +1 for overruling lack on test coverage in this case. I will merge once the rest of the travis jobs pass


-- 
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-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org