impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Impala Public Jenkins (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5383: Fix PARQUET FILE SIZE option for ADLS
Date Wed, 31 May 2017 07:41:25 GMT
Impala Public Jenkins has submitted this change and it was merged.

Change subject: IMPALA-5383: Fix PARQUET_FILE_SIZE option for ADLS


PARQUET_FILE_SIZE query option doesn't work with ADLS because the
AdlFileSystem doesn't have a notion of block sizes. And impala depends
on the filesystem remembering the block size which is then used as the
target parquet file size (this is done for Hdfs so that the parquet file
size and block size match even if the parquet_file_size isn't a valid

We special case for Adls just like we do for S3 to bypass the
FileSystem block size, and instead just use the requested
PARQUET_FILE_SIZE as the output partitions block_size (and consequently
the parquet file target size).

Testing: Re-enabled test_insert_parquet_verify_size() for ADLS.

Also fixed a miscellaneous bug with the ADLS client listing helper function.

Change-Id: I474a913b0ff9b2709f397702b58cb1c74251c25b
Reviewed-by: Sailesh Mukil <>
Tested-by: Impala Public Jenkins
M be/src/exec/
M be/src/util/
M be/src/util/hdfs-util.h
M tests/query_test/
M tests/util/
5 files changed, 17 insertions(+), 8 deletions(-)

  Impala Public Jenkins: Verified
  Sailesh Mukil: Looks good to me, approved

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: I474a913b0ff9b2709f397702b58cb1c74251c25b
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Sailesh Mukil <>

View raw message