[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2016-02-25 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-189099153 @runyontr thanks for standing up! > Given that I can fix the list above, how do I update this pull request with my repository? It's not really p

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2016-02-25 Thread runyontr
Github user runyontr commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-189093591 @felixcheung I'll be happy to take this up. From what I can see there is the following left: - Change string creation to use StringBuilder - Th

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2016-02-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-186915981 Ping. Would anyone else like to pick this up if we don't hear back from @raj-bains ? --- If your project is set up for it, you can reply to this email a

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2016-02-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-178406139 @raj-bains any update? --- 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 do

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2016-01-09 Thread raj-bains
Github user raj-bains commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-170313023 I'm on vacation for 10 days, then I'll update the patch and check this in --- If your project is set up for it, you can reply to this email and have your r

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2016-01-05 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-168963864 @raj-bains could you please elaborate on status of this PR - do you want to update\add doc\merge it or close it unresolved? --- If your project is set up for i

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2015-09-10 Thread raj-bains
Github user raj-bains commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-139374312 The request is ready to merge. Made changes based on the comments including adding documentation. --- If your project is set up for it, you can reply to t

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2015-09-03 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-137605638 Most probably you also want to update ` conf/zeppelin-site.xml.template` property `zeppelin.interpreters` with name of the new interpreter class --- If your pro

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2015-09-03 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-137373722 Thank you for great contribution! Can you please also provide the documentation for new interpreter (needs to be in a separate PR to gh-pages branch)?

[GitHub] incubator-zeppelin pull request: Add File Interpreter, HDFS Interp...

2015-09-03 Thread raj-bains
GitHub user raj-bains opened a pull request: https://github.com/apache/incubator-zeppelin/pull/276 Add File Interpreter, HDFS Interpreter and Tests This is the first version of File Interpreter that adds basic ls, cd and pwd functionality against WebHDFS. It addresses ZEPPELIN-198