[GitHub] [doris] WinkerDu commented on a diff in pull request #18834: [feature](multi-catalog) Rename multi-catalog config 'specified_database_list' to 'include_database_list', and introduce new multi-catalog config 'exclude_database_list'

["WinkerDu (via GitHub)" <gi...@apache.org>]

WinkerDu commented on code in PR #18834:
URL: https://github.com/apache/doris/pull/18834#discussion_r1173807309


##########
docs/en/docs/lakehouse/multi-catalog/hive.md:
##########
@@ -56,10 +56,13 @@ CREATE CATALOG hive PROPERTIES (
 
  In addition to `type` and  `hive.metastore.uris` , which are required, you can specify other parameters regarding the connection.
 
-> `specified_database_list`:
-> 
-> only synchronize the specified databases, split with ','. Default values is '' will synchronize all databases. db name is case sensitive.
-> 
+> `include_database_list`:

Review Comment:
   OK, I'll move docs in hive.md and iceberg.md to multi-catalog.md.
   And I think we should keep jdbc.md since config like `include_database_list` should work with `only_specified_database`



-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org