drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-5326) Unit tests failures related to the SERVER_METADTA
Date Wed, 08 Mar 2017 00:04:38 GMT

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

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

Github user laurentgo commented on a diff in the pull request:

    https://github.com/apache/drill/pull/775#discussion_r104813282
  
    --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/metadata/ServerMetaProvider.java
---
    @@ -76,7 +76,7 @@
           .setReadOnly(false)
           .setGroupBySupport(GroupBySupport.GB_UNRELATED)
           .setLikeEscapeClauseSupported(true)
    -      .setNullCollation(NullCollation.NC_AT_END)
    +      .setNullCollation(NullCollation.NC_HIGH)
    --- End diff --
    
    I'm not sure if the last sentence is correct. If user specified DESC, and null collation
is unspecified, then null should sort first, no (if always HIGH). NC_HIGH seems then the correct
value (unless the default can be changed in the planner configuration).


> Unit tests failures related to the SERVER_METADTA
> -------------------------------------------------
>
>                 Key: DRILL-5326
>                 URL: https://issues.apache.org/jira/browse/DRILL-5326
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Metadata
>    Affects Versions: 1.10.0
>            Reporter: Vitalii Diravka
>            Assignee: Vitalii Diravka
>            Priority: Blocker
>             Fix For: 1.10.0
>
>
> 1. In DRILL-5301 a new SERVER_META rpc call was introduced. The server will support this
method only from 1.10.0 drill version. For drill 1.10.0-SNAPHOT it is disabled. 
> When I enabled this method (by way of upgrading drill version to 1.10.0 or 1.11.0-SNAPSHOT)
I found the following exception:
> {code}
> java.lang.AssertionError: Unexpected/unhandled MinorType value GENERIC_OBJECT
> {code}
> It appears in several tests (for example in DatabaseMetadataTest#testNullsAreSortedMethodsSaySortedHigh).
> The reason of it is "GENERIC_OBJECT" RPC-/protobuf-level type is appeared in the ServerMetadata#ConvertSupportList.
(Supporting of GENERIC_OBJECT was added in DRILL-1126).
> The proposed solution is to add the appropriate "JAVA_OBJECT" sql type name for this
"GENERIC_OBJECT" RPC-/protobuf-level data type.
> 2. After fixing the first one the mentioned above test still fails by reason of the incorrect
"NullCollation" value in the "ServerMetaProvider". According to the [doc|https://drill.apache.org/docs/order-by-clause/#usage-notes]
the default val should be NC_HIGH (NULL is the highest value).



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message