[GitHub] zeppelin issue #2508: [ZEPPELIN-2817] Support default interpreter setting in...

2017-09-03 Thread reminia
Github user reminia commented on the issue:

https://github.com/apache/zeppelin/pull/2508
  
@zjffdu  @Leemoonsoo can you help review? 


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


[GitHub] zeppelin issue #2508: [ZEPPELIN-2817] Support default interpreter setting in...

2017-08-10 Thread reminia
Github user reminia commented on the issue:

https://github.com/apache/zeppelin/pull/2508
  
I decide to leave the api no change at now.  Maybe another jira should be 
created to address this. I've found that several create operation api doesnt 
return 201 but 200, such as repo add, paragraph add. @felixcheung @Leemoonsoo 


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


[GitHub] zeppelin issue #2508: [ZEPPELIN-2817] Support default interpreter setting in...

2017-08-09 Thread reminia
Github user reminia commented on the issue:

https://github.com/apache/zeppelin/pull/2508
  
Test added @felixcheung 


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


[GitHub] zeppelin issue #2508: [ZEPPELIN-2817] Support default interpreter setting in...

2017-08-06 Thread reminia
Github user reminia commented on the issue:

https://github.com/apache/zeppelin/pull/2508
  
@felixcheung @Leemoonsoo  can you help review this?


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


[GitHub] zeppelin issue #2508: [ZEPPELIN-2817] Support default interpreter setting in...

2017-07-29 Thread reminia
Github user reminia commented on the issue:

https://github.com/apache/zeppelin/pull/2508
  
Add doc and some modify. @felixcheung @sravan-s , plz help review again.
Some details:

1. post interpreter name not id, cause name is user friendly and id is 
something like CRB87NS
2. if the interpreter name doesnt exist,  return **bad request status** and 
no note is created.
3. if successfully created the note, return CREATED status  like the old 
DOC says.
4. name/interpreter/paragraphs no need to be supplied together, empty 
message is also ok.


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