bugfix, hopefully correct this time

Anselme Vignon · Anselme Vignon · commit 7c829cbcef3b · 2015-02-20T12:48:27.000+01:00
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTableOperations.scala b/sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTableOperations.scala
@@ -150,9 +150,9 @@ case class ParquetTableScan(
               // Parquet will leave partitioning columns empty, so we fill them in here.
             var i = 0
             while (i < requestedPartitionOrdinals.size) {
-            row (requestedPartitionOrdinals (i)._2) =
-            partitionRowValues (requestedPartitionOrdinals (i)._1)
-            i += 1
+              row (requestedPartitionOrdinals (i)._2) =
+              partitionRowValues (requestedPartitionOrdinals (i)._1)
+              i += 1
             }
             row
             }
@@ -164,6 +164,7 @@ case class ParquetTableScan(
                     .update(
                       requestedPartitionOrdinals (i)._2,
                       partitionRowValues (requestedPartitionOrdinals (i)._1))
+                  i += 1
                 }
                 Row.fromSeq(rVals)
               }

Original file line number	Diff line number	Diff line change
`@@ -150,9 +150,9 @@ case class ParquetTableScan(`
`150`	`150`	`// Parquet will leave partitioning columns empty, so we fill them in here.`
`151`	`151`	`var i = 0`
`152`	`152`	`while (i < requestedPartitionOrdinals.size) {`
`153`		`- row (requestedPartitionOrdinals (i)._2) =`
`154`		`- partitionRowValues (requestedPartitionOrdinals (i)._1)`
`155`		`- i += 1`
	`153`	`+ row (requestedPartitionOrdinals (i)._2) =`
	`154`	`+ partitionRowValues (requestedPartitionOrdinals (i)._1)`
	`155`	`+ i += 1`
`156`	`156`	`}`
`157`	`157`	`row`
`158`	`158`	`}`
`@@ -164,6 +164,7 @@ case class ParquetTableScan(`
`164`	`164`	`.update(`
`165`	`165`	`requestedPartitionOrdinals (i)._2,`
`166`	`166`	`partitionRowValues (requestedPartitionOrdinals (i)._1))`
	`167`	`+ i += 1`
`167`	`168`	`}`
`168`	`169`	`Row.fromSeq(rVals)`
`169`	`170`	`}`