Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/968#discussion_r143715206
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/util/VectorUtil.java ---
    @@ -42,7 +43,12 @@ public static void 
showVectorAccessibleContent(VectorAccessible va, final String
         System.out.println(rows + " row(s):");
         List<String> columns = Lists.newArrayList();
         for (VectorWrapper<?> vw : va) {
    -      columns.add(vw.getValueVector().getField().getName());
    +      MaterializedField field = vw.getValueVector().getField();
    --- End diff --
    
    Maybe we can factor out this block of code (I've seen it at least three 
times)?


---

Reply via email to