nileshrathi345 commented on code in PR #529:
URL: https://github.com/apache/livy/pull/529#discussion_r3482757368
##########
server/src/main/scala/org/apache/livy/server/batch/BatchSession.scala:
##########
@@ -137,6 +138,7 @@ object BatchSession extends Logging {
m.owner,
m.proxyUser,
sessionStore,
+ None,
Review Comment:
Currently, queue is primarily used during session creation and tracking.
However, passing None during recover means that if the Livy server restarts,
any recovered batch session will permanently lose its original queue context.
To maintain that have added to the state in previous version
--
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]