rymurr commented on a change in pull request #7289:
URL: https://github.com/apache/arrow/pull/7289#discussion_r433118490
##########
File path:
java/vector/src/main/java/org/apache/arrow/vector/complex/AbstractStructVector.java
##########
@@ -196,21 +245,51 @@ protected void putChild(String name, FieldVector vector) {
putVector(name, vector);
}
+ private void put(String name, FieldVector vector, boolean overwrite) {
+ final boolean old = vectors.put(
+ Preconditions.checkNotNull(name, "field name cannot be null"),
+ Preconditions.checkNotNull(vector, "vector cannot be null"),
+ overwrite
+ );
+ if (old) {
+ logger.debug("Field [{}] mutated to [{}] ", name,
+ vector.getClass().getSimpleName());
+ }
+ }
+
/**
* Inserts the input vector into the map if it does not exist, replaces if
it exists already.
Review comment:
fixed, thanks!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
