This is an automated email from the ASF dual-hosted git repository. twalthr pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/flink.git.
from e2ecd7a [FLINK-21800][core] Guard MemorySegment against concurrent frees. new 4497e96 [FLINK-21822][table] Support the new factory stack for CatalogFactory new b0fa390 [FLINK-21822][table] Migrate built-in and test catalog factories to new stack new a3e2ae4 [FLINK-21822][table] Introduce CatalogFactoryHelper new ddfcce7 [hotfix][core] Fix typos in ReadableConfig The 4 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: docs/content/docs/dev/table/catalogs.md | 7 +- .../flink/connectors/hive/HiveTableSink.java | 4 +- .../flink/connectors/hive/HiveTableSource.java | 4 +- .../flink/table/catalog/hive/HiveCatalog.java | 4 +- .../hive/descriptors/HiveCatalogDescriptor.java | 71 ---- .../hive/descriptors/HiveCatalogValidator.java | 39 --- .../catalog/hive/factories/HiveCatalogFactory.java | 97 ++---- .../hive/factories/HiveCatalogFactoryOptions.java | 51 +++ .../org.apache.flink.table.factories.Factory | 2 +- .../org.apache.flink.table.factories.TableFactory | 1 - .../descriptors/HiveCatalogDescriptorTest.java | 53 --- .../hive/factories/HiveCatalogFactoryTest.java | 85 +++-- .../jdbc/catalog/factory/JdbcCatalogFactory.java | 77 ++--- .../catalog/factory/JdbcCatalogFactoryOptions.java | 30 +- .../table/descriptors/JdbcCatalogDescriptor.java | 67 ---- .../table/descriptors/JdbcCatalogValidator.java | 41 --- .../org.apache.flink.table.factories.Factory | 1 + .../org.apache.flink.table.factories.TableFactory | 1 - .../catalog/factory/JdbcCatalogFactoryTest.java | 26 +- .../descriptors/JdbcCatalogDescriptorTest.java | 61 ---- .../apache/flink/configuration/ReadableConfig.java | 4 +- .../table/client/config/entries/CatalogEntry.java | 9 +- .../context/LegacyTableEnvironmentInitializer.java | 18 +- .../gateway/context/ExecutionContextTest.java | 21 +- .../table/client/gateway/local/DependencyTest.java | 66 ++-- .../client/gateway/local/EnvironmentTest.java | 4 +- .../client/gateway/utils/SimpleCatalogFactory.java | 64 ++-- ...ry => org.apache.flink.table.factories.Factory} | 5 +- .../org.apache.flink.table.factories.TableFactory | 4 - .../src/test/resources/sql/catalog_database.q | 3 +- .../src/test/resources/sql/function.q | 2 +- .../test/resources/test-sql-client-catalogs.yaml | 6 +- .../table/api/internal/TableEnvironmentImpl.java | 12 +- .../catalog/GenericInMemoryCatalogFactory.java | 60 ++-- .../GenericInMemoryCatalogFactoryOptions.java | 19 +- .../GenericInMemoryCatalogDescriptor.java | 42 --- .../GenericInMemoryCatalogValidator.java | 30 -- ...ry => org.apache.flink.table.factories.Factory} | 0 .../catalog/GenericInMemoryCatalogFactoryTest.java | 19 +- .../GenericInMemoryCatalogDescriptorTest.java | 64 ---- .../flink/table/catalog/CommonCatalogOptions.java | 28 +- .../flink/table/descriptors/CatalogDescriptor.java | 86 ----- .../descriptors/CatalogDescriptorValidator.java | 45 --- .../flink/table/factories/CatalogFactory.java | 95 +++++- .../apache/flink/table/factories/FactoryUtil.java | 193 ++++++++++- .../flink/table/factories/TableFactoryService.java | 3 +- .../table/descriptors/CatalogDescriptorTest.java | 103 ------ .../flink/table/factories/FactoryUtilTest.java | 55 +++- .../flink/table/factories/TestCatalogFactory.java | 364 +++++++++++++++++++++ .../org.apache.flink.table.factories.Factory | 1 + .../flink/table/planner/catalog/CatalogITCase.java | 57 +++- .../table/factories/CatalogFactoryServiceTest.java | 22 +- ...gFactory.java => TestLegacyCatalogFactory.java} | 26 +- .../org.apache.flink.table.factories.TableFactory | 2 +- 54 files changed, 1168 insertions(+), 1086 deletions(-) delete mode 100644 flink-connectors/flink-connector-hive/src/main/java/org/apache/flink/table/catalog/hive/descriptors/HiveCatalogDescriptor.java delete mode 100644 flink-connectors/flink-connector-hive/src/main/java/org/apache/flink/table/catalog/hive/descriptors/HiveCatalogValidator.java create mode 100644 flink-connectors/flink-connector-hive/src/main/java/org/apache/flink/table/catalog/hive/factories/HiveCatalogFactoryOptions.java copy {flink-table/flink-table-runtime-blink/src/test => flink-connectors/flink-connector-hive/src/main}/resources/META-INF/services/org.apache.flink.table.factories.Factory (92%) delete mode 100644 flink-connectors/flink-connector-hive/src/test/java/org/apache/flink/table/catalog/hive/descriptors/HiveCatalogDescriptorTest.java copy flink-kubernetes/src/main/java/org/apache/flink/kubernetes/configuration/KubernetesConfigOptionsInternal.java => flink-connectors/flink-connector-jdbc/src/main/java/org/apache/flink/connector/jdbc/catalog/factory/JdbcCatalogFactoryOptions.java (51%) delete mode 100644 flink-connectors/flink-connector-jdbc/src/main/java/org/apache/flink/table/descriptors/JdbcCatalogDescriptor.java delete mode 100644 flink-connectors/flink-connector-jdbc/src/main/java/org/apache/flink/table/descriptors/JdbcCatalogValidator.java delete mode 100644 flink-connectors/flink-connector-jdbc/src/test/java/org/apache/flink/table/descriptors/JdbcCatalogDescriptorTest.java copy flink-table/flink-sql-client/src/test/resources/META-INF/services/{org.apache.flink.table.factories.TableFactory => org.apache.flink.table.factories.Factory} (83%) copy flink-kubernetes/src/main/java/org/apache/flink/kubernetes/configuration/KubernetesConfigOptionsInternal.java => flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/catalog/GenericInMemoryCatalogFactoryOptions.java (61%) delete mode 100644 flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/descriptors/GenericInMemoryCatalogDescriptor.java delete mode 100644 flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/descriptors/GenericInMemoryCatalogValidator.java rename flink-table/flink-table-api-java/src/main/resources/META-INF/services/{org.apache.flink.table.factories.TableFactory => org.apache.flink.table.factories.Factory} (100%) delete mode 100644 flink-table/flink-table-api-java/src/test/java/org/apache/flink/table/descriptor/GenericInMemoryCatalogDescriptorTest.java copy flink-kubernetes/src/main/java/org/apache/flink/kubernetes/configuration/KubernetesConfigOptionsInternal.java => flink-table/flink-table-common/src/main/java/org/apache/flink/table/catalog/CommonCatalogOptions.java (54%) delete mode 100644 flink-table/flink-table-common/src/main/java/org/apache/flink/table/descriptors/CatalogDescriptor.java delete mode 100644 flink-table/flink-table-common/src/main/java/org/apache/flink/table/descriptors/CatalogDescriptorValidator.java delete mode 100644 flink-table/flink-table-common/src/test/java/org/apache/flink/table/descriptors/CatalogDescriptorTest.java create mode 100644 flink-table/flink-table-common/src/test/java/org/apache/flink/table/factories/TestCatalogFactory.java rename flink-table/flink-table-planner/src/test/java/org/apache/flink/table/factories/utils/{TestCatalogFactory.java => TestLegacyCatalogFactory.java} (77%)