Copilot commented on code in PR #10245: URL: https://github.com/apache/gravitino/pull/10245#discussion_r2888664443
########## docs/iceberg-rest-service.md: ########## @@ -19,9 +19,11 @@ There are some key difference between Gravitino Iceberg REST server and Gravitin ### Capabilities -- Supports the Apache Iceberg REST API defined in Iceberg 1.10, and supports all namespace and table interfaces. The following interfaces are not implemented yet: +- Supports the Apache Iceberg REST API defined in Iceberg 1.10, and supports most namespace, table and view interfaces. The following interfaces are not implemented yet: Review Comment: Minor grammar/clarity: “supports most namespace, table and view interfaces” reads awkwardly and uses singular “namespace”. Consider “supports most namespace, table, and view interfaces” -> “supports most namespaces, table, and view interfaces”, and avoid the repeated “supports … and supports …” phrasing for readability. ```suggestion - Supports the Apache Iceberg REST API defined in Iceberg 1.10 and most namespaces, table, and view interfaces. The following interfaces are not implemented yet: ``` -- 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]
