You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "JingsongLi (via GitHub)" <gi...@apache.org> on 2026/06/20 13:58:45 UTC
[PR] [core] Add option for catalog options system table [paimon]
JingsongLi opened a new pull request, #8303:
URL: https://github.com/apache/paimon/pull/8303
## Summary
Add a catalog option to control whether the `sys.catalog_options` system table is available. The table is disabled by default and can be enabled explicitly with `catalog-options-table.enabled=true`.
## Changes
- Added `catalog-options-table.enabled` to catalog options.
- Filtered `sys.catalog_options` from global system table listings unless the option is enabled.
- Guarded direct loading of `sys.catalog_options` with the same option.
- Updated docs and generated catalog configuration reference.
- Adjusted core, Flink, and Spark tests for the new default behavior.
## Testing
- `mvn -pl paimon-core -am -Pfast-build -DfailIfNoTests=false -Dtest=CatalogOptionsTableTest test`
- `mvn -pl paimon-core -am -Pfast-build -DfailIfNoTests=false -Dtest=FileSystemCatalogTest#testGetTable test`
- `mvn -pl paimon-spark/paimon-spark-ut -am -Pfast-build -DfailIfNoTests=false -DwildcardSuites=org.apache.paimon.spark.table.PaimonSystemTableTest -Dtest=none test`
- `mvn -pl paimon-flink/paimon-flink-common -am -Pfast-build -DfailIfNoTests=false -Dtest=CatalogTableITCase#testCatalogOptionsTable,CatalogTableITCase#testSystemDatabase test`
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [core] Add option for catalog options system table [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on code in PR #8303:
URL: https://github.com/apache/paimon/pull/8303#discussion_r3457043745
##########
paimon-core/src/main/java/org/apache/paimon/rest/RESTCatalog.java:
##########
@@ -241,6 +241,12 @@ public PagedList<String> listTablesPaged(
@Nullable String tableType)
throws DatabaseNotExistException {
try {
+ if (isSystemDatabase(databaseName)) {
+ CatalogUtils.validateNamePattern(this, tableNamePattern);
+ CatalogUtils.validateTableType(this, tableType);
+ return new PagedList<>(
Review Comment:
The new system-database path accepts the paged-list parameters but then returns the full global system table list with a null next token. Since `RESTCatalog` advertises support for paging, name patterns, and table-type filtering, calls such as `listTablesPaged("sys", 1, null, null, null)` or `listTablesPaged("sys", null, null, "catalog_%", null)` now return unpaged/unfiltered results instead of following the REST catalog contract. Could we either apply the same filtering/paging to `loadGlobalTableNames(...)` here and in `listTableDetailsPaged`, or reject these parameters for the system 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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [core] Add option for catalog options system table [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8303:
URL: https://github.com/apache/paimon/pull/8303
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org