Github user echarles commented on the issue:

    https://github.com/apache/zeppelin/pull/1390
  
    @zjffdu Correct, this PR also implements ZEPPELIN-1338 User level 
interpreter setting (I have updated the title and description on this PR).
    
    @Leemoonsoo Will check and adapt if needed the behavior for multiple owners 
and will further digg into #1265 to assess any conflit/divergence. I'd like to 
second the comment made by @zjffdu in #1265 on the interpreter options 
complexity. Btw, at the time of the introduction of the current interpreter 
modes, I already raised my worries on the complexity of those and honestly I am 
not convinced that I understand them fully (at least at first sight without 
digging into the code) and I fear that #1265 will make the situation still more 
complex. Taking back @zjffdu statement: *Besides, I think we should think more 
deeply on the relationship between note, user and interpreter.*, and I would 
add also the relation with the SearchService, Helium... and any other services 
that Zeppelin uses (cc/ @jongyoul) - All these questions also binds to 
ZEPPELIN-1236 "Multi-user notebook with user controls support" created by 
@frosiere talking about shared dashboard... Having this discussion on the ma
 iling list independently of any PR would better define the efforst to be taken 
in each PR.
    
    @khalidhuseynov I have the bad habit to build with `-Dcheckstyle.skip=true` 
- I commit a fix in the next hour.



---
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