impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5383: Fix PARQUET FILE SIZE option for ADLS
Date Tue, 30 May 2017 19:04:09 GMT
Sailesh Mukil has uploaded a new change for review.

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
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, 15 insertions(+), 6 deletions(-)

  git pull ssh:// refs/changes/18/7018/1
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newchange
Gerrit-Change-Id: I474a913b0ff9b2709f397702b58cb1c74251c25b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <>

View raw message