Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16086
Change subject: IMPALA-9831: Fix off by one error in condition for ValidateColumnOffsets() ...................................................................... IMPALA-9831: Fix off by one error in condition for ValidateColumnOffsets() ParquetMetadataUtils::ValidateColumnOffsets() returns an error if the end of the column is beyond the end of the file (i.e. offset > end_of_file). Instead, because there is a footer, the end of column must not be the end of the file either, so it should use offset >= end_of_file. Otherwise, a subsequent DCHECK in ParquetPageReader using the stricter condition will fire. Testing: - Core job Change-Id: I16bd6dfbb8eeacc1cb854ed4a3c2ed9f1c3aa11f --- M be/src/exec/parquet/parquet-metadata-utils.cc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/86/16086/1 -- To view, visit http://gerrit.cloudera.org:8080/16086 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I16bd6dfbb8eeacc1cb854ed4a3c2ed9f1c3aa11f Gerrit-Change-Number: 16086 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>