zhongyujiang opened a new pull request, #3143: URL: https://github.com/apache/parquet-java/pull/3143
<!-- Thanks for opening a pull request! If you're new to Parquet-Java, information on how to contribute can be found here: https://parquet.apache.org/docs/contribution-guidelines/contributing Please open a GitHub issue for this pull request: https://github.com/apache/parquet-java/issues/new/choose and format pull request title as below: GH-${GITHUB_ISSUE_ID}: ${SUMMARY} or simply use the title below if it is a minor issue: MINOR: ${SUMMARY} --> ### Rationale for this change When reading encrypted columns, if the provided key is NULL, the thrown exception message might be confusing: "Column [foo] is encrypted with NULL column key". However NULL key is certainly not permitted for encryption, the occurrence of such an exception invariably indicates an issue with the key provisioning process. So this PR explicitly states this in the exception message. ### What changes are included in this PR? Exception msg change. ### Are these changes tested? No need. ### Are there any user-facing changes? No. <!-- Please uncomment the line below and replace ${GITHUB_ISSUE_ID} with the actual Github issue id. --> <!-- Closes #${GITHUB_ISSUE_ID} --> @wgtmac @ggershinsky can you please take a look? Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@parquet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@parquet.apache.org For additional commands, e-mail: issues-h...@parquet.apache.org
