[GitHub] [incubator-iceberg] chenjunjiedada commented on issue #598: Support recognizing the column represented with an array of fields

[GitBox <gi...@apache.org>]

chenjunjiedada commented on issue #598: Support recognizing the column represented with an array of fields
URL: https://github.com/apache/incubator-iceberg/issues/598#issuecomment-548656440
 
 
   It makes sense if Iceberg has the restriction. I checked that when adding a column, the restriction works. While I can still define a SCHEMA like below and use it to create a table:
   ```java 
     Schema SIMPLE_SCHEMA = new Schema(
         required(1, "id", Types.IntegerType.get()),
         optional(2, "data", Types.StringType.get()),
         required(3, "a", Types.StructType.of(
             required(4, "b", Types.StructType.of(
                 required(5, "c", Types.BooleanType.get())
             )),
             required(6, "b.c", Types.BooleanType.get())
         ), "struct of named boolean options")
     );
   ```
   Do we need a validator for schema? Though I think no one would intend to define schema like this. Or I may miss some checks.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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