ujc714 opened a new pull request #2319: URL: https://github.com/apache/hive/pull/2319
### What changes were proposed in this pull request? Changed one line in service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java. The session log directory should be <operationLogRootDir>/<session_id>/ instead of <operationLogRootDir>/<sessionHandle_id>/. ### Why are the changes needed? If we run beeline/hive command with parameter "--hiveconf hive.session.id=xxx", we can't see the query operation logs. And the operation logs can't be cleaned up by HS2. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Added testSessionId1() and testSessionId2() in TestHiveCli.java. Run "mvn test -Dtest=org.apache.hive.beeline.cli.TestHiveCli" -- 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]
