nevzheng commented on code in PR #17138:
URL: https://github.com/apache/iceberg/pull/17138#discussion_r3562674387
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2787,10 +2787,9 @@ components:
FunctionReference:
description: >
- A reference to a function per Iceberg Expressions spec, Appendix B.
- Four JSON forms: a bare name (single-part identifier), a list of
- names (multi-part identifier), an object with an identifier, or an
- object with both a catalog and an identifier.
+ A reference to a function. Four JSON forms: a bare name (single-part
Review Comment:
I notice "JSON forms" across the description. It feels a bit clunky to me.
I would try to establish a consistent convention across descriptions. `X is
a/either/or x, y, z` (something like this maybe). and try to refer to the parts
with meaning.
ex: `A function reference is either a name, a multi-part name, an identifier
object, or a catalog-qualified identifier object. `
the goal is to concise.
--
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]
