Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1060#discussion_r158183647
  
    --- 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;
    --- End diff --
    
    Vectors commonly use the `MaterializedField` class rather than the 
over-the-wire `SerializedField`.


---

Reply via email to