[GitHub] zeppelin pull request #2206: [ZEPPELIN-1856] Flaky Test: InterpreterRestApiT...

2018-06-26 Thread AlexanderShoshin
Github user AlexanderShoshin closed the pull request at: https://github.com/apache/zeppelin/pull/2206 ---

[GitHub] zeppelin issue #2206: [ZEPPELIN-1856] Flaky Test: InterpreterRestApiTest.tes...

2017-06-06 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2206 @jongyoul it looks like I need to correct my unit tests due last changes in master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #2362: [ZEPPELIN-2469] HeliumBundleFactoryTest fails on Travi...

2017-05-22 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2362 We also need to enable the `HeliumBundleFactoryTest` on CI by removing it from `-Dtests.to.exclude` variable in `.travis.yml` --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #2300: [ZEPPELIN-2467] Restoring second travis job functional...

2017-05-12 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2300 I think we should revert this commit then. I won't be able to fix it fast. --- 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 #2300: [ZEPPELIN-2467] Restoring second travis job functional...

2017-05-03 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2300 According to @Leemoonsoo suggestion I have added a comment for the travis job configuration about excluded tests. --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin pull request #2300: [ZEPPELIN-2467] Restoring second travis job fun...

2017-04-28 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/2300 [ZEPPELIN-2467] Restoring second travis job functionality ### What is this PR for? Second travis job didn't work correctly. It suppose to test all core unit tests plus integr

[GitHub] zeppelin issue #2206: [ZEPPELIN-1856] Flaky Test: InterpreterRestApiTest.tes...

2017-04-27 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2206 Hi @jongyoul, I've added a test for my changes but it won't run until we restore second Travis job which should start all core tests. I will write when the test will

[GitHub] zeppelin issue #2233: [MINOR] Add paragraph to note

2017-04-17 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2233 This PR is ready for 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

[GitHub] zeppelin pull request #2246: [ZEPPELIN-1828] Flaky Test: RemoteInterpreterTe...

2017-04-12 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/2246 [ZEPPELIN-1828] Flaky Test: RemoteInterpreterTest hanged in zeppelin-interpreter ### What is this PR for? When `interpreter.close()` occurs in code it shuts down

[GitHub] zeppelin issue #2233: [MINOR] Add paragraph to note

2017-04-10 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2233 Hi Felix, thank you for looking through my PR. Method name must correspond an action it performs. Methods with names like `addSomething()` usually mean that we must pass this

[GitHub] zeppelin pull request #2233: [MINOR] Add paragraph to note

2017-04-07 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/2233 [MINOR] Add paragraph to note ### What is this PR for? We need to add an ability of adding custom _Paragraph_ to a _Note_. This will make it easier to write _Note_ and _Paragraph_

[GitHub] zeppelin issue #2206: [ZEPPELIN-1856] Flaky Test: InterpreterRestApiTest.tes...

2017-04-06 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2206 Sure, I'll do it. But first I need to make some refactoring of `Note.java` class in separate PR. We need to be able to add a custom `Paragraph` to `Note`. This will allow to

[GitHub] zeppelin pull request #2206: [ZEPPELIN-1856] Flaky Test: InterpreterRestApiT...

2017-03-30 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/2206 [ZEPPELIN-1856] Flaky Test: InterpreterRestApiTest.testRestartInterpreterPerNote ### What is this PR for? A _NullPointerException_ occurs occasionally in

[GitHub] zeppelin pull request #2188: [MINOR] Job parameters synchronization

2017-03-24 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/2188 [MINOR] Job parameters synchronization ### What is this PR for? We need to synchronize setting up job result, exception, errorMessage and dateFinished in the end of `Job.run

[GitHub] zeppelin issue #1765: [WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .pyS...

2017-02-20 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1765 I had similar issue a week ago: log - https://api.travis-ci.org/jobs/201058125/log.txt?deansi=true config - https://travis-ci.org/apache/zeppelin/jobs/201058125/config

[GitHub] zeppelin issue #2014: [ZEPPELIN-2065] Flaky test: NotebookTest.testScheduleP...

2017-02-13 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2014 One travis job failed - looks like known bag https://issues.apache.org/jira/browse/ZEPPELIN-1761 --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2014: [ZEPPELIN-2065] Flaky test: NotebookTest.testScheduleP...

2017-02-13 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2014 I changed AtomicInteger to CountDownLatch to simplify the code. --- 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

[GitHub] zeppelin pull request #2014: [ZEPPELIN-2065] Flaky test: NotebookTest.testSc...

2017-02-13 Thread AlexanderShoshin
Github user AlexanderShoshin closed the pull request at: https://github.com/apache/zeppelin/pull/2014 --- 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

[GitHub] zeppelin pull request #2014: [ZEPPELIN-2065] Flaky test: NotebookTest.testSc...

2017-02-13 Thread AlexanderShoshin
GitHub user AlexanderShoshin reopened a pull request: https://github.com/apache/zeppelin/pull/2014 [ZEPPELIN-2065] Flaky test: NotebookTest.testSchedulePoolUsage ### What is this PR for? `testSchedulePoolUsage()` test checks that jobs can be executed several times in a row

[GitHub] zeppelin pull request #2014: [ZEPPELIN-2065] Flaky test: NotebookTest.testSc...

2017-02-13 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/2014 [ZEPPELIN-2065] Flaky test: NotebookTest.testSchedulePoolUsage ### What is this PR for? `testSchedulePoolUsage()` test checks that jobs can be executed several times in a row

[GitHub] zeppelin pull request #1954: [ZEPPELIN-525] Test failing in zeppelin-interpr...

2017-01-27 Thread AlexanderShoshin
Github user AlexanderShoshin closed the pull request at: https://github.com/apache/zeppelin/pull/1954 --- 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

[GitHub] zeppelin pull request #1954: [ZEPPELIN-525] Test failing in zeppelin-interpr...

2017-01-27 Thread AlexanderShoshin
GitHub user AlexanderShoshin reopened a pull request: https://github.com/apache/zeppelin/pull/1954 [ZEPPELIN-525] Test failing in zeppelin-interpreter `RemoteSchedulerTest.test` was fall when CPU switched `RemoteScheduler` execution (which executes a job in a separate thread) back

[GitHub] zeppelin pull request #1954: [ZEPPELIN-525] Test failing in zeppelin-interpr...

2017-01-27 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/1954 [ZEPPELIN-525] Test failing in zeppelin-interpreter `RemoteSchedulerTest.test` was fall when CPU switched `RemoteScheduler` execution (which executes a job in a separate thread) back to

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2017-01-11 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 @AhyoungRyu, thanks for the note. I've removed an url to the notebook. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2017-01-09 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 I've added a new commit to convert the notebook to 0.7.0 format because I found that [9013620](https://github.com/apache/zeppelin/pull/1758/co

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2017-01-09 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 Sorry, I was not able to work on this issue during last weeks. @AhyoungRyu, I will make the corrections soon. Thanks --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-20 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 @bzz, I am not sure. We already have a *Word Count* example there to describe `Flink` usage. Where should we place the link to this example? --- If your project is set up for it, you

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-20 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 I am confused a bit :) @AhyoungRyu, do I need to drop my last commit or not? --- 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 #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-16 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 I've converted notebook to 0.6.2 format. So it should be displayed correctly now. [ZeppelinHub vew](https://www.zeppelinhub.com/viewer/note

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-15 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 Oh, it should not looks like this. It might be another problem of new notebook json structure. I will correct this. --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-15 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 @zjffdu, should we do this in current PR or make a new issue for the documentation improving? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-15 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 @tae-jun, I found that **ZeppelinHub** can't display new notebooks correctly. It is because the note.json structure was changed after [ZEPPELIN-212](https://issues.apache.org/jira/b

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-15 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 @AhyoungRyu, thank you for your notes! * I've changed `wget` to `curl` as you suggested. * As for the error message with "permission denied" I think it is because `

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-14 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/1758 Yes, but ZeppelinHub does not show all the paragraphs. I will try to find out the reason. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin pull request #1758: [ZEPPELIN-1787] Add an example of Flink Noteboo...

2016-12-14 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/1758 [ZEPPELIN-1787] Add an example of Flink Notebook ### What is this PR for? This PR will add an example of batch processing with Flink to Zeppelin tutorial notebooks. There are no any