[GitHub] zeppelin pull request #3264: Fix Date Sorting issue

[fred521 <gi...@git.apache.org>]

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 <fd...@...>
Date:   2018-09-25T01:01:23Z

    Fix Date Sorting issue

----


---

[GitHub] zeppelin issue #3264: [ZEPPELIN-3908] Fix Date Sorting issue

[felixcheung <gi...@git.apache.org>]

Github user felixcheung commented on the issue:

    https://github.com/apache/zeppelin/pull/3264
  
    please fix description - remove `First time? Setup Travis CI as described ` ...


---