[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r482457411 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r482452759 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r482443104 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r481466208 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -229,15 +290,258 @@ public void

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r481465744 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -229,15 +290,258 @@ public void

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r481334035 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -229,15 +279,259 @@ public void

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r481333253 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -229,15 +279,259 @@ public void

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r480763844 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -229,15 +279,259 @@ public void

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r479557601 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +279,255 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r479552231 ## File path: sdks/java/io/parquet/src/test/java/org/apache/beam/sdk/io/parquet/ParquetIOTest.java ## @@ -69,6 +76,44 @@ "Faraday", "Newton",

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r479551971 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +279,255 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r479551565 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +279,255 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467200308 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467200510 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -187,12 +229,18 @@ public Read

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467200389 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -187,12 +229,18 @@ public Read

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467199940 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467200061 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467199678 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467199561 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467199631 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467199486 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467199330 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467194486 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467164052 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467169175 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467168842 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467163612 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467161015 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467160391 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467159080 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r467158869 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,14 +281,284 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r464705615 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,12 +281,271 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r464701869 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -230,12 +281,271 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r464700678 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -177,6 +214,10 @@ public Read from(String

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r463259859 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +277,195 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r461928741 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +284,151 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r458297447 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +283,164 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r458293851 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +281,147 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r456821395 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +284,151 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r456821370 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +284,151 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r456820683 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +281,147 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r456818528 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +281,147 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r456673384 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +284,151 @@ public ReadFiles

[GitHub] [beam] danielxjd commented on a change in pull request #12223: [Beam-4379] Make ParquetIO read splittable

danielxjd commented on a change in pull request #12223: URL: https://github.com/apache/beam/pull/12223#discussion_r456673165 ## File path: sdks/java/io/parquet/src/main/java/org/apache/beam/sdk/io/parquet/ParquetIO.java ## @@ -235,12 +284,151 @@ public ReadFiles