[ https://issues.apache.org/jira/browse/DRILL-5326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15900414#comment-15900414 ]
ASF GitHub Bot commented on DRILL-5326: --------------------------------------- Github user vdiravka commented on a diff in the pull request: https://github.com/apache/drill/pull/775#discussion_r104812618 --- 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 -- @jinfengni Agree with Paul. The difference between NC_HIGH and NC_AT_END for the DESC case. I checked that Drill uses NC_HIGH for sorting. Please, see it at [jira](https://issues.apache.org/jira/browse/DRILL-5326?focusedCommentId=15900411&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15900411), since github doesn't show formatted code correctly in the comments. > 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)