[GitHub] zeppelin pull request #3264: Fix Date Sorting issue
GitHub user fred521 opened a pull request: https://github.com/apache/zeppelin/pull/3264 Fix Date Sorting issue ### What is this PR for? When plotting a histogram, Zeppelin does not order the x variable correctly, making the plot incorrect. I have attached screenshots of both histograms below. It seems that Zeppelin is putting numbers with no decimal places to the left (as if the were smaller) than numbers with decimal places. I tried casting the x axis value to a float but the problem persists.ns.html ### What type of PR is it? [Bug Fix] ### Todos N/A ### What is the Jira issue? [ZEPPELIN-3908] https://issues.apache.org/jira/browse/ZEPPELIN-3908 ### How should this be tested? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual steps to test the PR here. ### Screenshots (if appropriate) It is in the Jira ### Questions: * Does the licenses files need update? n/a * Is there breaking changes for older versions? n/a * Does this needs documentation? n/a You can merge this pull request into a Git repository by running: $ git pull https://github.com/fred521/zeppelin upstream/master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/3264.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 #3264 commit b3f9c6f6ea8902e74b54a540385e49d4586d3bf9 Author: fdai Date: 2018-09-25T01:01:23Z Fix Date Sorting issue ---
[GitHub] zeppelin issue #3262: [ZEPPELIN-3902]upgrade jetty version
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3262 My email daizhi...@gmail.com Actually I have done a lot of fixes this year. Trying to merge back now. Thanks again On Thu, Dec 13, 2018 at 21:29 Jongyoul Lee wrote: > @fred521 <https://github.com/fred521> BTW, Could you please tell me your > jira id or email address? I need to register you as a contributor > > â > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/zeppelin/pull/3262#issuecomment-447218231>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/AC9O8ZnsLZqMUjW77q32Dq-H8dRiZ-rwks5u4zcegaJpZM4ZQuKz> > . > ---
[GitHub] zeppelin issue #3262: [ZEPPELIN-3902]upgrade jetty version
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3262 Cool @jongyoul thanks, done with that please let me know if anything else I will need to follow > @fred521 Thanks. The last thing. Can you update your title including '[ZEPPELIN-3902]'? ---
[GitHub] zeppelin issue #3262: upgrade jetty version
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3262 > @fred521 I've found one of your tests failed. https://travis-ci.org/fred521/zeppelin/builds/467287988 Could you please check it again? Thanks seems randomly failed for the network issue. now it passed the CI, please see the link below. https://travis-ci.org/fred521/zeppelin/builds/467287988 ---
[GitHub] zeppelin issue #3254: [ZEPPELIN-3575] Add 'Copy Column Name' to table visual...
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3254 UI-Grid actually creates a lot of performance issue, why not deprecate it but still introducing new feature? In my service, I completely wiped out UI-Grid ---
[GitHub] zeppelin issue #3262: upgrade jetty version
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3262 Hi @jongyoul can you help to merge it? the CI passed Thanks ---
[GitHub] zeppelin issue #3262: upgrade jetty version
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3262 Thanks @felixcheung I have updated it ---
[GitHub] zeppelin pull request #3261: upgrade Jetty to be the latest version
Github user fred521 closed the pull request at: https://github.com/apache/zeppelin/pull/3261 ---
[GitHub] zeppelin pull request #3262: upgrade jetty version
GitHub user fred521 opened a pull request: https://github.com/apache/zeppelin/pull/3262 upgrade jetty version ### 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 - https://zeppelin.apache.org/contribution/contributions.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? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual 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/fred521/zeppelin fdai/upgradeJettyVersion Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/3262.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 #3262 commit cbb340d423a2652e248105ce092c4657455d72fd Author: fdai Date: 2018-12-06T21:29:17Z upgrade jetty version ---
[GitHub] zeppelin pull request #3260: upgrade jetty version
Github user fred521 closed the pull request at: https://github.com/apache/zeppelin/pull/3260 ---
[GitHub] zeppelin pull request #3261: upgrade Jetty to be the latest version
GitHub user fred521 opened a pull request: https://github.com/apache/zeppelin/pull/3261 upgrade Jetty to be the latest version ### 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 - https://zeppelin.apache.org/contribution/contributions.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? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual 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/fred521/zeppelin master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/3261.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 #3261 commit cbb340d423a2652e248105ce092c4657455d72fd Author: fdai Date: 2018-12-06T21:29:17Z upgrade jetty version commit db46b1e401419d1cddcd71989064f9a5b8ae6789 Author: Fred Dai Date: 2018-12-12T23:42:30Z Merge pull request #1 from fred521/fdai/upgradeJettyVersion upgrade jetty version ---
[GitHub] zeppelin pull request #3259: Upgrade Jetty version to the latest version - 9...
Github user fred521 closed the pull request at: https://github.com/apache/zeppelin/pull/3259 ---
[GitHub] zeppelin pull request #3260: upgrade jetty version
GitHub user fred521 opened a pull request: https://github.com/apache/zeppelin/pull/3260 upgrade jetty version ### 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 - https://zeppelin.apache.org/contribution/contributions.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? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual 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/fred521/zeppelin fdai/upgradeJettyVersion Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/3260.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 #3260 commit cbb340d423a2652e248105ce092c4657455d72fd Author: fdai Date: 2018-12-06T21:29:17Z upgrade jetty version ---
[GitHub] zeppelin issue #3259: Upgrade Jetty version to the latest version - 9.4.14.v...
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3259 here is the link https://ci.appveyor.com/project/ApacheSoftwareFoundation/zeppelin/builds/20912238 On Tue, Dec 11, 2018 at 6:37 PM Fred wrote: > Yes, It passes all the tests > > On Tue, Dec 11, 2018 at 6:31 PM Jongyoul Lee > wrote: > >> Could you please check your CI? I think this should pass all of our tests >> >> â >> You are receiving this because you authored the thread. >> Reply to this email directly, view it on GitHub >> <https://github.com/apache/zeppelin/pull/3259#issuecomment-446439022>, >> or mute the thread >> <https://github.com/notifications/unsubscribe-auth/AC9O8e34dbOrd6jwCOGJBLr0X759Acg1ks5u4GqfgaJpZM4ZMVYd> >> . >> > ---
[GitHub] zeppelin issue #3259: Upgrade Jetty version to the latest version - 9.4.14.v...
Github user fred521 commented on the issue: https://github.com/apache/zeppelin/pull/3259 Yes, It passes all the tests On Tue, Dec 11, 2018 at 6:31 PM Jongyoul Lee wrote: > Could you please check your CI? I think this should pass all of our tests > > â > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/apache/zeppelin/pull/3259#issuecomment-446439022>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/AC9O8e34dbOrd6jwCOGJBLr0X759Acg1ks5u4GqfgaJpZM4ZMVYd> > . > ---
[GitHub] zeppelin pull request #3259: Upgrade Jetty version to the latest version - 9...
GitHub user fred521 opened a pull request: https://github.com/apache/zeppelin/pull/3259 Upgrade Jetty version to the latest version - 9.4.14.v20181114 ### What is this PR for? Our service frequently failed because of exception from the websocket. Upgrade to the latest Jetty version to fix the hang issue if the websocket endpoint is invalid. 9.4.14.v20181114 https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.14.v20181114 9.4.13.v2018 https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.13.v2018 + 2875 Fix WebSocketClient.connect() hang when attempting to connect at an invalid websocket endpoint ### What type of PR is it? Bug Fix & Improvement ### Todos * [ ] - Task ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3902 Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-3902] ### How should this be tested? Tested in the dev server and production ### Screenshots (if appropriate) ### 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/fred521/zeppelin fdai/upgradeJettyVersion Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/3259.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 #3259 commit cbb340d423a2652e248105ce092c4657455d72fd Author: fdai Date: 2018-12-06T21:29:17Z upgrade jetty version ---
