You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2022/01/18 13:34:52 UTC

[GitHub] [drill] jnturton opened a new pull request #2430: DRILL-8023: Empty dict page breaks the "old" Parquet reader

jnturton opened a new pull request #2430:
URL: https://github.com/apache/drill/pull/2430


   # [DRILL-8023](https://issues.apache.org/jira/browse/DRILL-8023): Empty dict page breaks the "old" Parquet reader
   
   ## Description
   
   Dictionary pages of zero bytes caused exceptions to be thrown from the "old" Parquet reader which did not include logic to discard them.
   
   ## Documentation
   None.
   
   ## Testing
   TestEmptyParquet#testEmptyDictPage
   


-- 
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: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [drill] lgtm-com[bot] commented on pull request #2430: DRILL-8023: Empty dict page breaks the "old" Parquet reader

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #2430:
URL: https://github.com/apache/drill/pull/2430#issuecomment-1015503422


   This pull request **introduces 1 alert** when merging 1781dfc3b444c07eb6d596a69628732927fedc17 into 55e94c4e1c4a05ac7010391daea8f4f0804b0286 - [view on LGTM.com](https://lgtm.com/projects/g/apache/drill/rev/pr-09aabdde3d672d39317d77c0374f1eb95c0cbf67)
   
   **new alerts:**
   
   * 1 for Dereferenced variable may be null


-- 
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: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [drill] jnturton merged pull request #2430: DRILL-8023: Empty dict page breaks the "old" Parquet reader

Posted by GitBox <gi...@apache.org>.
jnturton merged pull request #2430:
URL: https://github.com/apache/drill/pull/2430


   


-- 
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: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org