Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1060#discussion_r158183682
--- Diff: exec/vector/src/main/codegen/templates/NullableValueVectors.java
---
@@ -15,13 +15,19 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
+import org.apache.drill.exec.proto.UserBitShared.SerializedField;
import org.apache.drill.exec.memory.AllocationManager.BufferLedger;
import org.apache.drill.exec.util.DecimalUtility;
import org.apache.drill.exec.vector.BaseDataValueVector;
import org.apache.drill.exec.vector.NullableVectorDefinitionSetter;
+import org.apache.drill.exec.vector.NullableVarCharVector.Accessor;
+import org.apache.drill.exec.vector.NullableVarCharVector.Mutator;
--- End diff --
All nullable vectors need the `VarChar` accessor and mutator?
---
