roryqi commented on code in PR #11808:
URL: https://github.com/apache/gravitino/pull/11808#discussion_r3490163743


##########
docs/iceberg-rest-engine/trino.md:
##########
@@ -190,6 +190,16 @@ WITH (
 
 ## Known Issues
 
+### `SHOW SCHEMAS` Fails on Gravitino IRC (OAuth2, Nested Namespaces)
+
+When Trino connects to Gravitino IRC with 
`iceberg.rest-catalog.security=OAUTH2`,
+`iceberg.rest-catalog.nested-namespace-enabled=true`, and 
`iceberg.rest-catalog.session=NONE`
+(the default), `SHOW SCHEMAS` recursively calls Iceberg REST `listNamespaces`. 
On Trino releases
+before 482, each recursive call creates a separate OAuth session, which can 
trigger excessive token
+requests and cause errors such as `Connection pool shut down` or 
`StackOverflowError`.
+

Review Comment:
   Could u refer to other official documents?
   Maybe it should better:
   Cause: xxx
   Solution: xxx.
   
   ``TIMESTAMP WITH TIME ZONE` Values Are Not Adjusted to the Client Session 
Time Zone`
   should refer to this format, too.
   
   Maybe I prefer
   
   Solution: 
   Upgrade to Trino 482+.
   



-- 
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]

Reply via email to