[GitHub] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...

[jongyoul]

Github user jongyoul commented on the issue:

    https://github.com/apache/zeppelin/pull/806
  
    It works well with another default interpreter except `%spark`. See the 
below.
    <img width="659" alt="screen shot 2016-06-10 at 6 05 11 pm" 
src="https://cloud.githubusercontent.com/assets/3612566/15959490/0720f9e6-2f36-11e6-9f8b-ad6f627c4582.png";>
    <img width="448" alt="screen shot 2016-06-10 at 6 05 22 pm" 
src="https://cloud.githubusercontent.com/assets/3612566/15959503/108bf364-2f36-11e6-98db-f8648d8f582e.png";>
    
    But I also don't think it's good idea to display `%..` by default. It would 
be better display actual default name of interpreter like `%spark` or `%jdbc`. 
It prevent new users from confusing and trying to understand what that means.
    
    @mwkang Could you please try to fix it? and OccupiedInterpreter looks 
redundant because it stores noteId-interpreterName map only and returns 
interpreterName with noteId. This can be parsed from InterpreterFactory or 
another class directly. What's the benefit to create OccupiedInterpreter class?
    
    I also think it's simple to choose default interpreter when we create 
notebook and store it into note.json and use it.


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