vrajat opened a new issue, #15231: URL: https://github.com/apache/pinot/issues/15231
In integration tests, brokers and servers run in the same JVM. All of them share the same `ThreadUsageResourceAccountant` in a singleton in `Tracing.java`. This makes certain kinds of tests hard to write. For example: * #14999 * Tests that involve relevant operations in a specific server. Some tests use `InterruptingAccountant`. This class is useful to inject errors in the broker. However, it is not possible to inject errors in servers as the broker will error out first. Couple of options are: * Run a server in a different JVM. This is a major change in how tests have to be written. It will also make debug mode harder. * Use dependency injection rather than a singleton to pass around an accountant object. -- 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...@pinot.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
