barronw commented on code in PR #40436:
URL: https://github.com/apache/arrow/pull/40436#discussion_r1524099630
##########
java/vector/src/main/java/org/apache/arrow/vector/NullVector.java:
##########
@@ -73,17 +74,19 @@ public NullVector(String name, int valueCount) {
* @param name name of the vector
* @param fieldType type of Field materialized by this vector.
*/
+ @Deprecated
public NullVector(String name, FieldType fieldType) {
- this(new Field(name, fieldType, null));
+ this(name);
}
/**
* Instantiate a NullVector.
*
* @param field field materialized by this vector.
*/
+ @Deprecated
public NullVector(Field field) {
Review Comment:
Thanks for the feedback!
Could you elaborate on why generic code would need the various field
constructors? I think an example would be helpful.
Are the field type constructors also needed?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
