[GitHub] incubator-zeppelin pull request: Zeppelin configuration getStringV...

2016-05-04 Thread onkarshedge
GitHub user onkarshedge opened a pull request: https://github.com/apache/incubator-zeppelin/pull/873 Zeppelin configuration getStringValue() returns multiple values comma… ### What is this PR for? ZeppelinConfiguration getStringValue(String name,String d) returns only a

[GitHub] incubator-zeppelin pull request: Zeppelin configuration getStringV...

2016-05-04 Thread onkarshedge
Github user onkarshedge commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/873#issuecomment-217082413 @bzz so shall I send another pull request with the test case in ZeppelinConfigurationTest.java or in this PR itself ? --- If your project is set up for

[GitHub] incubator-zeppelin pull request: Zeppelin configuration getStringV...

2016-05-05 Thread onkarshedge
Github user onkarshedge closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/873 --- 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

[GitHub] incubator-zeppelin pull request: Zeppelin configuration getStringV...

2016-05-05 Thread onkarshedge
Github user onkarshedge commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/873#issuecomment-217352893 Ohh, I get it. I was adding new NotebookRepos in zeppelin-site.xml this way. So I was getting only one Notebookrepo and wrote the code in PR to combine