[
https://issues.apache.org/jira/browse/DRILL-6613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16547921#comment-16547921
]
ASF GitHub Bot commented on DRILL-6613:
---------------------------------------
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]
> Refactor MaterializedField
> --------------------------
>
> Key: DRILL-6613
> URL: https://issues.apache.org/jira/browse/DRILL-6613
> Project: Apache Drill
> Issue Type: Improvement
> Reporter: Vlad Rozov
> Assignee: Vlad Rozov
> Priority: Minor
>
> {{MaterializedField}} does not need to implement {{clone()}} and should use
> constructor.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
