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

Posted to reviews@livy.apache.org by GitBox <gi...@apache.org> on 2019/02/14 00:47:41 UTC

[GitHub] alex-the-man edited a comment on issue #103: [LIVY-239] Generate session IDs in SessionStore, not in SessionManager

alex-the-man edited a comment on issue #103: [LIVY-239] Generate session IDs in SessionStore, not in SessionManager
URL: https://github.com/apache/incubator-livy/pull/103#issuecomment-463437562
 
 
   > > Can we add some migration code so user can seamlessly upgrade from older Livy? Thanks!
   > 
   > Should the migration code be part of Livy? Or should be a separate script that users run once?
   
   In this case it seems simple enough to be a part of the startup code.
   
   > 
   > Also should we change the version in zookeeper path from `/v1/...` to `/v2/...`?
   
   I think you are right, we should.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services