Github user anubhav100 commented on the issue:

    https://github.com/apache/carbondata/pull/1236
  
    @steven-qin
    1.please correct your PR title as per : 
https://github.com/apache/carbondata/blob/master/docs/How-to-contribute-to-Apache-CarbonData.md
    2.there is still problem with decimal type i tried to run this query in 
both carbon data and presto from your branch and presto  is giving wrong results
    
    in carbon-----
    
    carbon.sql("select l_tax from lineitem where l_tax=0.06").show()
        carbon.stop()
    
    +-----+
    |l_tax|
    +-----+
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    | 0.06|
    +-----+
    only showing top 20 rows
    
    when i execute it in presto 
    
    select l_tax from lineitem where l_tax=0.06;
    
    i get no result 



---
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.
---

Reply via email to