Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1393
  
    Thanks @kavinkumarks for the contribution.
    This changes not only resolving race condition described in ZEPPELIN-1142 
but also changes user experience. i.e.
    
    Before, owner is not defined when creating note.
    After, owner is set to current user when creating note. (when 
authentication is enabled)
    
    Especially with code changes such as 
https://github.com/apache/zeppelin/pull/1330, user will experience bigger 
behavior changes.
    
    But i think having current user as a owner when creating note is natural 
behavior that most people can think, even though it brings some UX changes.
    
    LGTM and merge if there're no further discussions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to