[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-09-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/22232 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-08-28 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/22232#discussion_r213543748 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileSourceSuite.scala --- @@ -0,0 +1,55 @@ +/* + * Licensed to the

[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-08-28 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22232#discussion_r213483961 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileSourceSuite.scala --- @@ -0,0 +1,55 @@ +/* + * Licensed to the

[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-08-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/22232#discussion_r212796191 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala --- @@ -208,7 +199,6 @@ class FileScanRDD(

[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-08-25 Thread dujunling
Github user dujunling commented on a diff in the pull request: https://github.com/apache/spark/pull/22232#discussion_r212793909 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala --- @@ -208,7 +199,6 @@ class FileScanRDD(

[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-08-25 Thread dujunling
Github user dujunling commented on a diff in the pull request: https://github.com/apache/spark/pull/22232#discussion_r212793866 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala --- @@ -208,7 +199,6 @@ class FileScanRDD(

[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-08-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/22232#discussion_r212793049 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala --- @@ -208,7 +199,6 @@ class FileScanRDD(

[GitHub] spark pull request #22232: [SPARK-25237][SQL]remove updateBytesReadWithFileS...

2018-08-25 Thread dujunling
GitHub user dujunling opened a pull request: https://github.com/apache/spark/pull/22232 [SPARK-25237][SQL]remove updateBytesReadWithFileSize because we use Hadoop FileSystem s… …tatistics to update the inputMetrics ## What changes were proposed in this pull request?