[
https://issues.apache.org/jira/browse/TAJO-1257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14283535#comment-14283535
]
Hudson commented on TAJO-1257:
------------------------------
ABORTED: Integrated in Tajo-master-CODEGEN-build #198 (See
[https://builds.apache.org/job/Tajo-master-CODEGEN-build/198/])
TAJO-1257: ORDER BY with NULL FIRST misses some data. (Keuntae Park) (sirpkt:
rev 4d1006d56b7761500e16afe7be7826422850555a)
* tajo-core/src/test/java/org/apache/tajo/engine/query/TestSortQuery.java
* CHANGES
* tajo-core/src/main/java/org/apache/tajo/engine/utils/TupleUtil.java
> ORDER BY with NULL FIRST misses some data
> -----------------------------------------
>
> Key: TAJO-1257
> URL: https://issues.apache.org/jira/browse/TAJO-1257
> Project: Tajo
> Issue Type: Bug
> Reporter: Keuntae Park
> Assignee: Keuntae Park
> Priority: Minor
> Fix For: 0.10
>
>
> Even with TAJO-904 and TAJO-1239, ORDER BY NULL FIRST misses some data.
> Below test code doesn't pass.
> It seems that this problem comes from misbehaving setMaxRangeIfNull() method
> in TupleUtil.java, which places NULL in the range by only considering the
> existence of ASC/DESC.
> {code}
> KeyValueSet tableOptions = new KeyValueSet();
> tableOptions.set(StorageConstants.TEXT_DELIMITER,
> StorageConstants.DEFAULT_FIELD_DELIMITER);
> tableOptions.set(StorageConstants.TEXT_NULL, "\\\\N");
> Schema schema = new Schema();
> schema.addColumn("id", Type.INT4);
> schema.addColumn("name", Type.TEXT);
> String[] data = new String[]{ "1|111", "2|\\N", "3|333" };
> TajoTestingCluster.createTable("table11", schema, tableOptions, data, 1);
> try {
> ResultSet res = executeString("select * from table11 order by name null
> first");
> String ascExpected = "id,name\n" +
> "-------------------------------\n" +
> "2,null\n" +
> "1,111\n" +
> "3,333\n";
> assertEquals(ascExpected, resultSetToString(res));
> res.close();
> } finally {
> executeString("DROP TABLE table11 PURGE");
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
