Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread 임정택
Hi moon soo, Thanks for the information. I'll try to have a look into websocket implementation. Regarding creating a notebook, currently NewNotebookRequest has just an one field, "name", so I think it is not supported now. Please correct me if I'm wrong. I'm working on this issue, and I'll try to

[GitHub] incubator-zeppelin pull request: Interpreter properties editor upd...

2015-12-14 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/525#issuecomment-164673854 @corneadoug thanks for the review. Removed the table header style and other changes in de75e49 --- If your project is set up for it, you can reply to this

Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread moon soo Lee
Hi Jungtaek Lim, I think run REST API with changing value of dynamic form make sense. Run API implementation of websocket server already does the job. So +1 for having ability to changing value of dynamic form from the run REST API. Doesn't it already have a paragraph when creating a notebook vi

[GitHub] incubator-zeppelin pull request: ZEPPELIN-500 - Fix leak of output...

2015-12-14 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/531#issuecomment-164660342 Merging if there is no more 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

[GitHub] incubator-zeppelin pull request: Drag to resize paragraph

2015-12-14 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/516#issuecomment-164659904 Found plenty of issues: 1) Scrollbar when code line is too long on notebook refresh/load ![screen shot 2015-12-15 at 3 29 52 pm](https://cloud

[GitHub] incubator-zeppelin pull request: Replace standard alert and confir...

2015-12-14 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-164659614 > Neither the cancel nor the OK button has focus on them. Would be nice to have either one pre-focussed. @gauravkumar37 thanks for the feedback. I've cre

[GitHub] incubator-zeppelin pull request: Drag to resize paragraph

2015-12-14 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/516#issuecomment-164651768 Could you also un-commit the note.json? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-12-14 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-164651464 @hsaputra There shouldn't be any changes in the new branch that are relevant to Travis CI testing. If there's anything I can do to move the Travis issue for

[GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-12-14 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-164649022 Could you try to create Travis job for your fork repository to test the changes? Shall this PR be closed in favor on new PR from the new branch? -

Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread 임정택
Yeah, actually that's what I thought it could be recognized as bad. It could make another confusion between the REST API and the UI, but someone can say it is acceptable since we already know which paragraphs we're trying to run via REST API. If we agree that it is acceptable, I'll work on it and

[GitHub] incubator-zeppelin pull request: Improve/split paragraph html

2015-12-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/324 --- 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 fea

Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread Corneau Damien
I guess the run paragraph REST API fall short on that point. When the UI run a paragraph query, it sends to the backend an object with more informations (like the form values). So it should be possible to extend the current REST API call to accept options (in order to override the original paragrap

[jira] [Created] (ZEPPELIN-501) Ability to search code in notebooks

2015-12-14 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-501: --- Summary: Ability to search code in notebooks Key: ZEPPELIN-501 URL: https://issues.apache.org/jira/browse/ZEPPELIN-501 Project: Zeppelin Issue

[GitHub] incubator-zeppelin pull request: Drag to resize paragraph

2015-12-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/516#issuecomment-164643672 I have tested and it works really well. Thanks @prabhjyotsingh for really improvement. One thing is, resize drag handle is shown in report mode. B

[GitHub] incubator-zeppelin pull request: Interpreter properties editor upd...

2015-12-14 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/525#issuecomment-164642819 I don't like the new table style, because I think that grey background doesn't go well with the rest. Originally the header of the table component is e

Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread 임정택
Corneau, Actual use case I'm trying to accomplish is a bit simple. One notebook contains three paragraphs. 1. load fat jar which contains the code (dump contents of HBase table to HDFS) 2. set the variables (table name, hdfs path, etc.) which the code will use > I'm using dynamic form at this pa

[GitHub] incubator-zeppelin pull request: Remove unnecessary spaces from do...

2015-12-14 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/533 Remove unnecessary spaces from docs/manual/dynamicform.md #518 introduces some unnecessary spaces in docs/manual/dynamicform.md which results dynamicform.md is not compiled to dynamic

Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread Corneau Damien
Maybe if you could provide some actual use case (type of code, what you want to replace, what do you want as output, where do you want that output, what is the goal), it could help us giving you better advices. On Tue, Dec 15, 2015 at 1:09 PM, 임정택 wrote: > Thanks Corneau, > > I'm using dynamic

Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread 임정택
Thanks Corneau, I'm using dynamic form already. At the first glance, exposing field and getting value to / from dynamic form seems to be up to interpreter, so I'm wondering I can set the form's value via unified way. And I'm curious that we can accomplish this without affecting any other contexts

Re: Way to run notebook via REST API with dynamic content

2015-12-14 Thread Corneau Damien
I don't know about the Run paragraph API, but the option 2 should already be possible: https://zeppelin.incubator.apache.org/docs/0.6.0-incubating-SNAPSHOT/manual/dynamicform.html On Tue, Dec 15, 2015 at 12:37 PM, 임정택 wrote: > Hi Zeppelin dev fellows, > > I was finding a way to run notebook vi

Way to run notebook via REST API with dynamic content

2015-12-14 Thread 임정택
Hi Zeppelin dev fellows, I was finding a way to run notebook via REST API with dynamic variable, and seems like there's no way to get it. I found three ways to achieve that functionality (Sure there could be more), please see below. 1. provide a way to update (I mean replace) content of the para

[GitHub] incubator-zeppelin pull request: JDBC generic interpreter

2015-12-14 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-164620557 And could you please check https://www.mail-archive.com/dev@zeppelin.incubator.apache.org/msg05504.html at dev@. I, strongly, think every contribution shoul

[GitHub] incubator-zeppelin pull request: JDBC generic interpreter

2015-12-14 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-164620107 @vgmartinez Sorry for delay. I've committed #455 #517 for supporting multiple instances - connections - Because that's based on this interpreter, it would b

[GitHub] incubator-zeppelin pull request: ZEPPELIN-490 Tidying up dynamicfo...

2015-12-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/518 --- 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 fea

[GitHub] incubator-zeppelin pull request: ZEPPELIN-496 ] Bug fixed Spark sc...

2015-12-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/529 --- 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 fea

[GitHub] incubator-zeppelin pull request: [WIP] fix bootstrap dialog defaul...

2015-12-14 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/532 [WIP] fix bootstrap dialog default button ### What is this PR for? Fix default focus issue on bootstrap alert/confirm box ### What type of PR is it? Improvement

Web application build error

2015-12-14 Thread Dave Cardon
Hi, I'm trying to get zeppelin to build OOB on a minimal CentOS 7 install. I followed the README.md instructions. My first snag was an issue getting bower installed, which I was able to work past by manually installing bower globally (npm -g install bower) prior to running the maven install.

[GitHub] incubator-zeppelin pull request: Replace standard alert and confir...

2015-12-14 Thread gauravkumar37
Github user gauravkumar37 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-164472215 Sorry to barge in a closed bug. Neither the cancel nor the OK button has focus on them. Would be nice to have either pre-focussed. --- If your pro

[GitHub] incubator-zeppelin pull request: ZEPPELIN-408 Fix configuration bu...

2015-12-14 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/420#issuecomment-164449569 @chiwanpark Sorry for late reply. I think it would be a kind of flaky test. I'll test this in my local machine. --- If your project is set up for it, you

[GitHub] incubator-zeppelin pull request: ZEPPELIN-500 - Fix leak of output...

2015-12-14 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/531#issuecomment-164407281 LGTM --- 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 fe

[GitHub] incubator-zeppelin pull request: ZEPPELIN-500 - Fix leak of output...

2015-12-14 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/531#issuecomment-164407115 @gauravkumar37 Create an issue since its not related to this PR. also include as much info regarding how to reproduce it. --- If your project is set u

[GitHub] incubator-zeppelin pull request: ZEPPELIN-500 - Fix leak of output...

2015-12-14 Thread gauravkumar37
Github user gauravkumar37 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/531#issuecomment-164406026 @corneadoug Thanks for the fix. Tested successfully on latest version of Chrome (v47), Firefox (v42) & IE11. I don't have Opera/Safari though. This

[GitHub] incubator-zeppelin pull request: Drag to resize paragraph

2015-12-14 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/516#issuecomment-164396499 Have updated this PR, instead of having multiple resize handles. Merged it into one. ![drag to resize paragraph](https://cloud.githubusercon