[
https://issues.apache.org/jira/browse/DRILL-6570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16528581#comment-16528581
]
ASF GitHub Bot commented on DRILL-6570:
---------------------------------------
kkhatua commented on issue #1354: DRILL-6570: Fixed IndexOutofBoundException in
Parquet Reader
URL: https://github.com/apache/drill/pull/1354#issuecomment-401522121
@sachouche The change looks simple, but I might be missing the bigger
picture. For e.g., `ColumnPrecisionInfo.precision` is passed as an unused
argument. Not clear as to why you would not need it still.
Perhaps someone else who is more familiar could review this.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> IndexOutOfBoundsException when using Flat Parquet Reader
> ---------------------------------------------------------
>
> Key: DRILL-6570
> URL: https://issues.apache.org/jira/browse/DRILL-6570
> Project: Apache Drill
> Issue Type: Bug
> Components: Storage - Parquet
> Reporter: salim achouche
> Assignee: salim achouche
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.14.0
>
> Original Estimate: 2h
> Remaining Estimate: 2h
>
> * The Parquet Reader creates a reusable bulk entry based on the column
> precision
> * It uses the column precision for optimizing the intermediary heap buffers
> * It first detected the column was fixed length but then it reverted this
> assumption when the column changed precision
> * This step was fine except the bulk entry memory requirement changed though
> the code didn't update the bulk entry intermediary buffers
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
