jackye1995 commented on code in PR #6612: URL: https://github.com/apache/iceberg/pull/6612#discussion_r1072544944
########## format/view-spec.md: ########## @@ -107,7 +107,7 @@ This type of representation stores the original view definition in SQL and its S | Required/Optional | Field Name | Description | |-------------------|------------|-------------| | Required | type | A string indicating the type of representation. It is set to "sql" for this type. | -| Required | sql | A string representing the original view definition in SQL | +| Required | query | A string representing the original view definition in SQL | Review Comment: why not change the API to `sql()`? I think it feels more consistent there since the API class is `SQLViewRepresentation` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
