[ 
https://issues.apache.org/jira/browse/DRILL-6447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16508101#comment-16508101
 ] 

ASF GitHub Bot commented on DRILL-6447:
---------------------------------------

arina-ielchiieva closed pull request #1291: DRILL-6447: Fixed a sanity check 
condition
URL: https://github.com/apache/drill/pull/1291
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git 
a/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/PageReader.java
 
b/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/PageReader.java
index bf75695b6b..a047fcc3e3 100644
--- 
a/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/PageReader.java
+++ 
b/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/PageReader.java
@@ -445,8 +445,8 @@ public void clear(){
    * @throws IOException An IO related condition
    */
   void resetDefinitionLevelReader(int skipCount) throws IOException {
-    if (parentColumnReader.columnDescriptor.getMaxDefinitionLevel() != 0) {
-      throw new UnsupportedOperationException("Unsupoorted Operation");
+    if (parentColumnReader.columnDescriptor.getMaxDefinitionLevel() > 1) {
+      throw new UnsupportedOperationException("Unsupported Operation");
     }
 
     final Encoding dlEncoding = 
METADATA_CONVERTER.getEncoding(pageHeader.data_page_header.definition_level_encoding);
diff --git 
a/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/VarLenBulkPageReader.java
 
b/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/VarLenBulkPageReader.java
index b6205c1ef3..394f6ff863 100644
--- 
a/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/VarLenBulkPageReader.java
+++ 
b/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/VarLenBulkPageReader.java
@@ -72,6 +72,7 @@
       this.pageInfo.numPageFieldsRead = pageInfoInput.numPageFieldsRead;
       this.pageInfo.definitionLevels = pageInfoInput.definitionLevels;
       this.pageInfo.dictionaryValueReader = 
pageInfoInput.dictionaryValueReader;
+      this.pageInfo.numPageValues = pageInfoInput.numPageValues;
     }
 
     this.columnPrecInfo = columnPrecInfoInput;
@@ -94,6 +95,7 @@ final void set(PageDataInfo pageInfoInput) {
     pageInfo.numPageFieldsRead = pageInfoInput.numPageFieldsRead;
     pageInfo.definitionLevels = pageInfoInput.definitionLevels;
     pageInfo.dictionaryValueReader = pageInfoInput.dictionaryValueReader;
+    pageInfo.numPageValues = pageInfoInput.numPageValues;
 
     buffer.clear();
   }


 

----------------------------------------------------------------
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]


> Unsupported Operation when reading parquet data
> -----------------------------------------------
>
>                 Key: DRILL-6447
>                 URL: https://issues.apache.org/jira/browse/DRILL-6447
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Storage - Parquet
>    Affects Versions: 1.14.0
>            Reporter: salim achouche
>            Assignee: Vlad Rozov
>            Priority: Major
>             Fix For: 1.14.0
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> An exception is thrown when reading Parquet data.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to