github-actions[bot] commented on code in PR #70176:
URL: https://github.com/apache/doris/pull/70176#discussion_r4487765296
##########
fe/fe-core/src/main/java/org/apache/doris/arrowflight/FlightSqlSchemaHelper.java:
##########
@@ -168,7 +168,7 @@ protected static byte[] getSerializedSchema(List<Field>
fields) {
*/
public void setParameterForGetTables(CommandGetTables command) {
includeSchema = command.getIncludeSchema();
- catalogFilterPattern = command.hasCatalog() ? command.getCatalog() :
"internal";
+ catalogFilterPattern = command.hasCatalog() ? command.getCatalog() :
null;
Review Comment:
[P2] Preserve raw external schema names for Flight metadata filters
When `show_full_dbname_in_info_schema_db` is enabled,
`FrontendServiceImpl.getDbNames()` matches and returns external names as
`catalog.db`. Because this helper now passes `dbSchemaFilterPattern` through
for the omitted/all-catalog path, `getSchemas(null, "multi_catalog")` is
compared with `hive_catalog.multi_catalog` and returns no schema; any emitted
`db_schema_name` is also qualified even though Flight already supplies
`catalog_name` separately. Normalize the pattern/output to the raw database
name (or use a names-only service path that does not apply the MySQL display
transformation) before exposing external catalogs.
--
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]
