vrozov commented on a change in pull request #1383: DRILL-6613: Refactor 
MaterializedField
URL: https://github.com/apache/drill/pull/1383#discussion_r203408822
 
 

 ##########
 File path: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/ColumnState.java
 ##########
 @@ -123,7 +123,7 @@ public static MapArrayColumnState 
build(ResultSetLoaderImpl resultSetLoader,
       // Create the map's offset vector.
 
       UInt4Vector offsetVector = new UInt4Vector(
-          BaseRepeatedValueVector.OFFSETS_FIELD,
+          MaterializedField.OFFSETS_FIELD,
 
 Review comment:
   It is not a name. It is an immutable static `MaterializedField` that can be 
shared between vectors. The name still belongs to `ValueVector`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to