[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-19 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r579325897 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala ## @@ -864,6 +864,73 @@ class

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-17 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r578041797 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -147,7 +150,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-17 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r578039796 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -147,7 +150,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-17 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r577844041 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -147,7 +150,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-08 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r572286331 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -147,7 +150,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-05 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r570430807 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -145,7 +146,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-04 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r570430807 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -145,7 +146,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-02 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r568799488 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala ## @@ -1205,6 +1205,32 @@ class

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-02 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r568824046 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala ## @@ -1205,6 +1205,32 @@ class

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-02-02 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r568799488 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala ## @@ -1205,6 +1205,32 @@ class

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-01-27 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r565853372 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala ## @@ -1196,6 +1196,24 @@ class

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-01-26 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r564886965 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -122,7 +123,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-01-26 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r564863081 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedRleValuesReader.java ## @@ -351,25 +351,43 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-01-25 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r563882773 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -122,7 +123,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-01-25 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r563882773 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -122,7 +123,24 @@ public

[GitHub] [spark] razajafri commented on a change in pull request #31284: [SPARK-34167][SQL]Reading parquet with IntDecimal written as a LongDecimal blows up

2021-01-22 Thread GitBox
razajafri commented on a change in pull request #31284: URL: https://github.com/apache/spark/pull/31284#discussion_r562830885 ## File path: sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java ## @@ -122,7 +123,19 @@ public