wombatu-kun commented on PR #19251: URL: https://github.com/apache/hudi/pull/19251#issuecomment-4958000029
Done in 746651b7bc2b. 1. Fixed here rather than deferred: `Option(super.createTable(...)).getOrElse(loadTable(ident))` now guards all three staging methods. The NPE does reproduce - with the guard reverted, the new test fails on `Cannot invoke "Table.schema()" because the return value of "BasicStagedTable.table()" is null`. Impact reworded. 2. Removed from the description. 3. Added `TestHoodieCatalogStagedTable`: a mock delegate installed via `setDelegateCatalog`, driving all three staging methods over both the delegation and the null-`createTable` path. It sits in `hudi-spark` because `stageCreate` reaches `sparkAdapter`, which only resolves against the version shim. 4. Dropped. -- 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]
