Hi I am working on https://issues.apache.org/jira/browse/BEAM-10497
The location string defined here[1] be Nullable ? If yes we should throw some exceptions while that location field is needed and not present like here[2] or [3] or [4] and there are yet some other places. Or should I make that NonNullable ? As I am still not acquainted to the code and new contributor, I am not aware of repercussions in either of the cases. [1] https://github.com/apache/beam/blob/v2.23.0-RC1/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/meta/Table.java#L40 [2] https://github.com/apache/beam/blob/v2.23.0-RC1/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/meta/provider/mongodb/MongoDbTable.java#L91 [3] https://github.com/apache/beam/blob/v2.23.0-RC1/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/meta/provider/text/TextTableProvider.java#L88 [4] https://github.com/apache/beam/blob/v2.23.0-RC1/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/meta/provider/avro/AvroTableProvider.java#L50 Regards Jayendra
