[GitHub] zeppelin pull request #2547: Fix trivial typo
GitHub user byung-u opened a pull request: https://github.com/apache/zeppelin/pull/2547 Fix trivial typo ### What is this PR for? Fixes a trivial typo in travis_check.py. The semicolon does nothing in travis_check.py. Python use semicolon when several statements on the same line, here is [python doc.](https://docs.python.org/3/reference/compound_stmts.html) ### Questions: * Does the licenses files need update? NO * Is there breaking changes for older versions? NO * Does this needs documentation? NO You can merge this pull request into a Git repository by running: $ git pull https://github.com/byung-u/zeppelin feature/fix-trivial-typo Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2547.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2547 commit fef77e51a85d4ff5dabcdc568fad43eb34fcb427 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-22T02:21:37Z feature: remove semicolon end of statement --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #2546: Fix trivial typo
Github user byung-u closed the pull request at: https://github.com/apache/zeppelin/pull/2546 --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #2546: Fix trivial typo
GitHub user byung-u opened a pull request: https://github.com/apache/zeppelin/pull/2546 Fix trivial typo ### What is this PR for? Fixes a trivial typo in travis_check.py. The semicolon does nothing in travis_check.py. Python use semicolon when several statements on the same line, here is [python doc](https://docs.python.org/3/reference/compound_stmts.html) ### What type of PR is it? [Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring] ### Todos * [ ] - Task ### What is the Jira issue? * Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/ * Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-533] ### How should this be tested? Outline the steps to test the PR here. ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? * Is there breaking changes for older versions? * Does this needs documentation? You can merge this pull request into a Git repository by running: $ git pull https://github.com/byung-u/zeppelin feature/trivial-typo Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2546.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2546 commit 9b9cfa1e2854dccddf6789c8e11c253c4697ec94 Author: Prabhjyot Singh <prabhjyotsi...@gmail.com> Date: 2017-08-10T23:27:28Z [ZEPPELIN-2846] Add selenium test case for AnyOfRolesAuthorizationFilter This is to test the new feature that was brought in with ZEPPELIN-2825 (org.apache.zeppelin.utils.AnyOfRolesAuthorizationFilter) [Improvement] * [ZEPPELIN-2846](https://issues.apache.org/jira/browse/ZEPPELIN-2846) CI should be green * Does the licenses files need update? N/A * Is there breaking changes for older versions? N/A * Does this needs documentation? N/A Author: Prabhjyot Singh <prabhjyotsi...@gmail.com> Closes #2524 from prabhjyotsingh/ZEPPELIN-2846 and squashes the following commits: e2a7ad548 [Prabhjyot Singh] add selenium test case for AnyOfRolesAuthorizationFilter commit d3ec39776805b0458ff26b9ff9f9a60f326b7155 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-17T08:00:06Z Merge branch 'master' of https://github.com/apache/zeppelin commit bf30e268b968187b19fe507a47d74e4491e4bcfc Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-18T01:11:21Z Merge branch 'master' of https://github.com/apache/zeppelin commit 9803e0810cf4bbb3dc0a381f7ee4a90f6705d465 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-17T02:56:27Z feature: fix last job execution time commit 8716e048c685b6ccb74479ce775ea05f2ce142b0 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-17T07:57:17Z Revert "feature: fix last job execution time" This reverts commit 180b951af61f16fbb54c1301cc8ccbc36347cd92. commit 8f987fd29bad7e554fb3f7aed48b35e24ad56bd1 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-17T08:09:01Z Fix last job execution time commit 392c5c44d77ee396020496582d95c35a80da3567 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-22T01:54:46Z Merge branch 'master' of https://github.com/apache/zeppelin commit b6e06f7ae3f44b54a607bab8df644022b3579d74 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-22T01:58:42Z feature: Fix trivial typo --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2543: [ZEPPELIN-2860] Invalid last job execution time
Github user byung-u commented on the issue: https://github.com/apache/zeppelin/pull/2543 @tae-jun Thank you for your comment. I'm gonna fix & test again. My code use only 1st paragraph for every note, because of this logic inside double for loop. If I have 1,000 notes and 1,000 paragraphs for every note, then `lastRunningUnixTime = getUnixTimeLastRunParagraph(paragraph);` is calling 1,000,000 times. ``` for (Note note : notes) { ... for (Paragraph paragraph : note.getParagraphs()) { ... lastRunningUnixTime = getUnixTimeLastRunParagraph(paragraph); } } ``` --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2543: [ZEPPELIN-2860] Invalid last job execution time
Github user byung-u commented on the issue: https://github.com/apache/zeppelin/pull/2543 @1ambda Thank you for helping me. --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2540: [ZEPPELIN-2860] Invalid last job execution time
Github user byung-u commented on the issue: https://github.com/apache/zeppelin/pull/2540 Move to #2543 --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #2540: [ZEPPELIN-2860] Invalid last job execution time
Github user byung-u closed the pull request at: https://github.com/apache/zeppelin/pull/2540 --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #2543: feature: fix last job execution time
GitHub user byung-u opened a pull request: https://github.com/apache/zeppelin/pull/2543 feature: fix last job execution time ### What is this PR for? Fixed invalid last job execution time ### What type of PR is it? [Bug Fix] ### What is the Jira issue? * [ZEPPELIN-2860](https://issues.apache.org/jira/browse/ZEPPELIN-2860) ### How should this be tested? Run Notebook and check job page execution time ### Screenshots (if appropriate) ![image](https://user-images.githubusercontent.com/16890077/29394147-74a35a42-8343-11e7-973f-0aee5acb81bc.png) ### Questions: * Does the licenses files need update? NO * Is there breaking changes for older versions? NO * Does this needs documentation? NO You can merge this pull request into a Git repository by running: $ git pull https://github.com/byung-u/zeppelin feature/fix-last-job-execution-time Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2543.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2543 commit 88d7f10b6760249cf36d82f39498a82a89c84f52 Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-17T02:56:27Z feature: fix last job execution time --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2540: [ZEPPELIN-2860] Invalid last job execution time
Github user byung-u commented on the issue: https://github.com/apache/zeppelin/pull/2540 Thank you for your review. I've tried to make CI green, but still fails though. It looks like r-base-dev package install fail. ![2017-08-16 14 37 36](https://user-images.githubusercontent.com/16890077/29348724-a9fe188e-8290-11e7-9bb2-74e732bf8c16.png) This issue also happened. * https://github.com/apache/zeppelin/pull/2445 --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2535: [ZEPPELIN-2860] Invalid last execution time
Github user byung-u commented on the issue: https://github.com/apache/zeppelin/pull/2535 duplicated to PR #2540 --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #2535: [ZEPPELIN-2860] Invalid last execution time
Github user byung-u closed the pull request at: https://github.com/apache/zeppelin/pull/2535 --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #2540: [ZEPPELIN-2860] Invalid last job execution time
GitHub user byung-u opened a pull request: https://github.com/apache/zeppelin/pull/2540 [ZEPPELIN-2860] Invalid last job execution time ### What is this PR for? Fixed invalid last job execution time on jobmanager page ### What type of PR is it? [Bug Fix] ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-2860 ### How should this be tested? Run Notebook and check jobmanager page execution time ### Screenshots (if appropriate) ![image](https://user-images.githubusercontent.com/16890077/29318945-179934c0-820d-11e7-86f4-fe5f46454f7b.png) ### Questions: * Does the licenses files need update? NO * Is there breaking changes for older versions? NO * Does this needs documentation? NO You can merge this pull request into a Git repository by running: $ git pull https://github.com/byung-u/zeppelin invalid_last_excution_time Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2540.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2540 commit bc20aca229ed55256e3ba5d894059d001eed371e Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-15T13:47:38Z Fix invalid last excution time --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #2535: Invalid last execution time
GitHub user byung-u opened a pull request: https://github.com/apache/zeppelin/pull/2535 Invalid last execution time ### What is this PR for? Fixed Invalid last execution time ### What type of PR is it? [Bug Fix] ### Todos ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-2860 ### How should this be tested? Run paragraph and check job page. ### Screenshots (if appropriate) ![image](https://user-images.githubusercontent.com/16890077/29306773-0a3acfc4-81da-11e7-901d-9392ca20b88d.png) ### Questions: * Does the licenses files need update? NO * Is there breaking changes for older versions? NO * Does this needs documentation? NO You can merge this pull request into a Git repository by running: $ git pull https://github.com/byung-u/zeppelin fix_job_time Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2535.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2535 commit a2a4a64efd39e6fd4fd1dc81c4f835f15772654c Author: byung-u <iam.byung...@gmail.com> Date: 2017-08-15T07:47:30Z Fix job manager time error --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---