eric-maynard commented on code in PR #1150:
URL: https://github.com/apache/polaris/pull/1150#discussion_r1990314136
##########
spec/polaris-catalog-service.yaml:
##########
@@ -127,13 +127,21 @@ paths:
# /v1/{prefix}/namespaces/{namespace}/tables/{table}/tasks:
# Not implemented in Polaris
- ######################
- # Polaris-native API #
- ######################
+ ###################################
+ # Polaris-native notification API #
Review Comment:
I think this was changed to make it clear the notifications don't relate to
generic tables (although I suppose they could). So in that way, you could say
it's good to have the names mismatch.
However I still think `Polaris Notification API` & `Polaris Generic Table
API` is fine. `Polaris-native Generic Table API` is a bit of a mouthful.
--
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]
