Gabriel39 opened a new pull request, #70176:
URL: https://github.com/apache/doris/pull/70176

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: None
   
   Problem Summary:
   
   Flight SQL `GetDbSchemas` and `GetTables` restrict an omitted catalog to 
`internal`, hiding external catalog metadata. An explicitly empty catalog has 
the opposite problem: the underlying Thrift API treats it as an unrestricted 
search.
   
   Preserve an omitted catalog as an unrestricted search, keep named catalog 
filters, and return no matches for an empty catalog because every Doris 
database belongs to a catalog. Request database names without IDs to avoid 
materializing database objects when discovering metadata across catalogs. 
Existing user identity and privilege filtering remain in use.
   
   ### Release note
   
   Flight SQL schema and table discovery includes external catalogs when the 
catalog argument is omitted. An empty catalog argument returns no matches.
   
   ### Check List (For Author)
   
   - [x] Unit Test: 48 related FE tests passed, including 12 new parameterized 
cases that fail on the original implementation. Coverage includes schema/table 
discovery, both include-schema modes, same-named databases across catalogs, 
serialized array schemas, and names-only requests.
   - [x] Regression test: add raw Flight client coverage comparing external 
catalog metadata with SQL SHOW/DESC results. The framework and new Groovy suite 
compile; execution requires the external test environment and was not performed 
locally.
   - Behavior changed: Yes, omitted and empty catalog arguments follow Flight 
SQL semantics; explicit named filters retain their behavior.
   - Does this need documentation: No, this corrects protocol compatibility.
   


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

Reply via email to