Now that the store*() contract has been stregthened, we can remove an…

… extra lookup
nemccarthy · Apr 22, 2015 · c754ae1 · c754ae1
1 parent f764d13
commit c754ae1
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/execution/UnsafeGeneratedAggregate.scala b/sql/core/src/main/scala/org/apache/spark/sql/execution/UnsafeGeneratedAggregate.scala
@@ -329,9 +329,6 @@ case class UnsafeGeneratedAggregate(
               PlatformDependent.LONG_ARRAY_OFFSET,
               emptyAggregationBuffer.length
             )
-            // So that the pointers point to the value we just stored:
-            // TODO: reset this inside of the map so that this extra looup isn't necessary
-            buffers.lookup(unsafeRowBuffer, PlatformDependent.LONG_ARRAY_OFFSET, keyLengthInBytes)
           }
           // Reset our pointer to point to the buffer stored in the hash map
           val address = loc.getValueAddress