Add todo.

cube2222 · Aug 7, 2023 · d49fbe7 · d49fbe7
1 parent 5372273
commit d49fbe7
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/arrowexec/nodes/group_by.go b/arrowexec/nodes/group_by.go
@@ -251,7 +251,7 @@ func (key *KeyInt) GetBatch(length int, offset int) arrow.Array {
 }
 
 type KeyString struct {
-	// TODO: Refactor to use custom array data.
+	// TODO: Use a LargeStringArray, instead of a StringArray.
 	// We use a custom set of buffers, rather than using the builder, because accessing elements of the builder is slow.
 	offsetsData     *memory.Buffer // TODO: Release these buffers at some point.
 	offsetsState    []int32        // This uses the above data as the storage underneath.