[ 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)