jlalwani-amazon opened a new pull request, #43:
URL: https://github.com/apache/flink-connector-hive/pull/43

   ## What is the purpose of the change
   
   Remove APIs deprecated by 
[FLIP-346](https://cwiki.apache.org/confluence/display/FLINK/FLIP-346:+Deprecate+ManagedTable+related+APIs).
 These were introduced for Apache Paimon (incubating) via FLIP-188 but are now 
obsolete — Paimon has moved to its own locking mechanism and no longer uses 
these interfaces.
   
   *JIRA: [FLINK-39655](https://issues.apache.org/jira/browse/FLINK-39655)*
   
   ## Brief change log
   
   - Delete `HiveCatalogLock.java`
   - Remove `RequireCatalogLock` usage from `HiveDynamicTableFactory`
   - Remove `ManagedTableListener.isManagedTable()` calls from `HiveCatalog` 
(replaced with `false`)
   - Remove `HiveCatalog.supportsManagedTable()` override
   - Remove `managedTable` parameter from 
`HiveTableUtil.instantiateHiveTable()` and `alterTableViaCatalogBaseTable()`
   - Delete `TestLockTableSinkFactory` and its SPI registration
   - Remove `testCatalogLock` and `testCreateAndGetFlinkManagedTable` tests
   
   ## Verifying this change
   
   CI passed on fork: 
https://github.com/jlalwani-amazon/flink-connector-hive/actions
   
   ```bash
   mvn compile test-compile -pl flink-connector-hive -am
   ```
   
   ## Does this pull request potentially affect one of the following parts?
   
   - Dependencies: **no**
   - The public API: **no** (removes deprecated APIs per FLIP-346)
   - The serializers: **no**
   - The runtime per-record code paths: **no**
   - Anything that affects deployment or recovery: **no**
   


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

Reply via email to