[GitHub] [iceberg] stevenzwu commented on issue #7507: API: StructProjection returns a null Projection object when the nested struct is null

["stevenzwu (via GitHub)" <gi...@apache.org>]

stevenzwu commented on issue #7507:
URL: https://github.com/apache/iceberg/issues/7507#issuecomment-1534976937

   if we have a schema like this
   ```
   Schema schema =
           new Schema(
               Types.NestedField.required(0, "id", Types.LongType.get()),
               Types.NestedField.optional(
                   3,
                   "location",
                   Types.StructType.of(
                       Types.NestedField.required(1, "lat", Types.FloatType.get()),
                       Types.NestedField.required(2, "long", Types.FloatType.get()))));
   ```
   
   when the nested location struct is null
   ```
   GenericRowData rowDataNullStruct = GenericRowData.of(1L, null);
   ```
   
   the StructProjection or Flink RowDataProjection would create a projection object wrapping a null struct value. I was proposing that it should just return a null projection object directly.
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   Message Russell Spitzer
   
   
   
   
   
   
   Shift + Return to add a new line
   Thread
   [commute-seattle](https://a1350286.slack.com/archives/CKACYTD51/p1683158954847069)
   
   
   
   Aswin Tekur Srinivasamurthy
     [15 hours ago](https://a1350286.slack.com/archives/CKACYTD51/p1683158954847069)
   [@Otto Melara](https://a1350286.slack.com/team/U02D7CNV9M5)
    
   [@Jesse Stanbridge](https://a1350286.slack.com/team/WNPHL3WFM)
    what’s the policy on using Lyft for late shuttles. Currently BL1710 is about 20 mins late
   :thread:
   1
   
   2 replies
   
   
   Aswin Tekur Srinivasamurthy
     [15 hours ago](https://a1350286.slack.com/archives/CKACYTD51/p1683158976676199?thread_ts=1683158954.847069&cid=CKACYTD51)
   IMG_0203
    
   IMG_0203
   
   
   
   
   Jesse Stanbridge
     [15 hours ago](https://a1350286.slack.com/archives/CKACYTD51/p1683159590854369?thread_ts=1683158954.847069&cid=CKACYTD51)
   The Lyft program only covers if a shuttle has overflowed and there's no room for you to board
   :ack:
   2
   
   
   
   
   
   
   
   
   
   
   
   
   Reply…
   Also send to commute-seattle


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org