zabetak opened a new pull request #1858: URL: https://github.com/apache/hive/pull/1858
### What changes were proposed in this pull request? 1. Add Jupiter extension for using mock TCP servers in tests 2. Add Jupiter extension proving LLAP daemons for unit tests 3. Add unit test reproducing the file descriptor leak with LLAP 4. Replace `LlapRoutingAppenderPurgePolicy` with `IdlePurgePolicy` ### Why are the changes needed? 1. Solve the descriptor leak problem. 2. Delegate the burden of managing appenders to the Log4j2 framework 3. Simplify code 4 New extensions are needed for making the unit test reproducer more readable ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? `mvn -pl llap-server test -Dtest=TestLlapDaemonLogging`, fails on f49bbc0dd9a270e7fc6472c43989f6e8f1590e50 `mvn -pl llap-server test -Dtest=TestLlapDaemonLogging`, passes on 458ac1467247f2a7c8275a5ab79a339c63b37e3c Tests exclusively for the Jupiter extension: `mvn -pl testutils test -Dtest=TestDoNothingTCPServerExtension` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
