[GitHub] zeppelin pull request #1882: [MINOR] Small Selenium test fixes

2017-01-10 Thread bitchelov
GitHub user bitchelov opened a pull request: https://github.com/apache/zeppelin/pull/1882 [MINOR] Small Selenium test fixes ### What is this PR for? Remove unused imports; Replace "Thread.sleep" (unstable solution); Add "deleteTestNotebook()"

[GitHub] zeppelin issue #1747: [MINOR] Remove unused import. Naming convention.

2017-01-09 Thread bitchelov
Github user bitchelov commented on the issue: https://github.com/apache/zeppelin/pull/1747 @lresende is it right? --- 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

[GitHub] zeppelin issue #1738: Strings should not be concatenated using '+' in a loop

2016-12-20 Thread bitchelov
Github user bitchelov commented on the issue: https://github.com/apache/zeppelin/pull/1738 @AhyoungRyu ok, ill update jira issue. --- 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

[GitHub] zeppelin issue #1738: Strings should not be concatenated using '+' in a loop

2016-12-20 Thread bitchelov
Github user bitchelov commented on the issue: https://github.com/apache/zeppelin/pull/1738 @Leemoonsoo https://issues.apache.org/jira/secure/ViewProfile.jspa?name=Sergey+Sokur --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #1747: Remove unused import. Naming convention.

2016-12-13 Thread bitchelov
Github user bitchelov commented on the issue: https://github.com/apache/zeppelin/pull/1747 @lresende Sure. --- 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

[GitHub] zeppelin pull request #1747: Minor changes

2016-12-12 Thread bitchelov
GitHub user bitchelov opened a pull request: https://github.com/apache/zeppelin/pull/1747 Minor changes ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide -

[GitHub] zeppelin pull request #1746: Collection.isEmpty() should be used to test for...

2016-12-12 Thread bitchelov
GitHub user bitchelov opened a pull request: https://github.com/apache/zeppelin/pull/1746 Collection.isEmpty() should be used to test for emptiness ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check ou

[GitHub] zeppelin pull request #1738: Small Refactoring

2016-12-08 Thread bitchelov
GitHub user bitchelov opened a pull request: https://github.com/apache/zeppelin/pull/1738 Small Refactoring ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide -