[ 
https://issues.apache.org/jira/browse/DRILL-6094?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16450078#comment-16450078
 ] 

ASF GitHub Bot commented on DRILL-6094:
---------------------------------------

Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1232#discussion_r183358609
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/store/parquet/TestVarlenDecimal.java
 ---
    @@ -17,61 +17,137 @@
      */
     package org.apache.drill.exec.store.parquet;
     
    +import org.apache.commons.io.FileUtils;
    +import org.apache.drill.exec.ExecConstants;
     import org.apache.drill.test.BaseTestQuery;
     import org.apache.drill.exec.planner.physical.PlannerSettings;
    -import org.apache.drill.exec.proto.UserBitShared;
    -import org.junit.BeforeClass;
    -import org.junit.AfterClass;
    +import org.hamcrest.CoreMatchers;
    +import org.junit.Assert;
     import org.junit.Test;
     
    -public class TestVarlenDecimal extends BaseTestQuery {
    -  // enable decimal data type
    -  @BeforeClass
    --- End diff --
    
    Why did you remove before and after class? Instead of you are setting 
`alterSession(PlannerSettings.ENABLE_DECIMAL_DATA_TYPE_KEY, true);` in tests...


> Decimal data type enhancements
> ------------------------------
>
>                 Key: DRILL-6094
>                 URL: https://issues.apache.org/jira/browse/DRILL-6094
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.12.0
>            Reporter: Volodymyr Vysotskyi
>            Assignee: Volodymyr Vysotskyi
>            Priority: Major
>              Labels: doc-impacting
>             Fix For: 1.14.0
>
>
> Currently, Decimal types are disabled by default since existing Decimal 
> implementation has a lot of flaws and performance problems. The goal of thisĀ 
> Jira to describe majority of them and possible ways of improving existing 
> implementation to be able to enable Decimal data types by default.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to