[DRILL-4989] Fix TestParquetWriter.testImpalaParquetBinaryAsTimeStamp_DictChange

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.9.0
• Assignee: Vitalii Diravka
• Reporter: Arina Ielchiieva
• Reviewer: Arina Ielchiieva
• Created: 02/Nov/16 13:10
• Updated: 02/Nov/16 18:53
• Resolved: 02/Nov/16 18:53

Description

Test TestParquetWriter.testImpalaParquetBinaryAsTimeStamp_DictChange expects sorted output but query does not include order by clause. This causes unit test to fail sometimes:

Expected Records near verification failure:
Record Number: 0 { `int96_ts` : 1970-01-01T00:00:01.000Z, }
Record Number: 1 { `int96_ts` : 1971-01-01T00:00:01.000Z, }
Record Number: 2 { `int96_ts` : 1972-01-01T00:00:01.000Z, }
Record Number: 3 { `int96_ts` : 1973-01-01T00:00:01.000Z, }
Record Number: 4 { `int96_ts` : 1974-01-01T00:00:01.000Z, }
Record Number: 5 { `int96_ts` : 2010-01-01T00:00:01.000Z, }
Record Number: 6 { `int96_ts` : 2011-01-01T00:00:01.000Z, }
Record Number: 7 { `int96_ts` : 2012-01-01T00:00:01.000Z, }
Record Number: 8 { `int96_ts` : 2013-01-01T00:00:01.000Z, }
Record Number: 9 { `int96_ts` : 2014-01-01T00:00:01.000Z, }

Actual Records near verification failure:
Record Number: 0 { `int96_ts` : 2010-01-01T00:00:01.000Z, }
Record Number: 1 { `int96_ts` : 2011-01-01T00:00:01.000Z, }
Record Number: 2 { `int96_ts` : 2012-01-01T00:00:01.000Z, }
Record Number: 3 { `int96_ts` : 2013-01-01T00:00:01.000Z, }
Record Number: 4 { `int96_ts` : 2014-01-01T00:00:01.000Z, }
Record Number: 5 { `int96_ts` : 2015-01-01T00:00:01.000Z, }
Record Number: 6 { `int96_ts` : 2016-01-01T00:00:01.000Z, }
Record Number: 7 { `int96_ts` : 1970-01-01T00:00:01.000Z, }
Record Number: 8 { `int96_ts` : 1971-01-01T00:00:01.000Z, }
Record Number: 9 { `int96_ts` : 1972-01-01T00:00:01.000Z, }

To fix add order by in select statement:

select int96_ts from dfs_test.`%s/parquet/int96_dict_change` order by int96_ts

Issue Links

• links to: GitHub Pull Request #643