csurong opened a new pull request, #4346:
URL: https://github.com/apache/amoro/pull/4346

   ## Why are the changes needed?
   
   Close #4345.
   
   When mixed-format discovery encounters an Iceberg table whose metadata file 
is missing, table loading aborts the entire namespace listing. The failure is 
then incorrectly reported as a missing database, which hides healthy tables and 
obscures the actual cause.
   
   ## Brief change log
   
   - Skip tables that cannot be loaded because their metadata is missing or the 
table was concurrently removed, while recording a warning.
   - Convert only `NoSuchNamespaceException` to `NoSuchDatabaseException` and 
preserve other failures.
   - Add regression tests for missing metadata, concurrent table removal, 
missing namespaces, and unexpected failures.
   
   ## How was this patch tested?
   
   - [x] Add test cases that check the changes thoroughly including negative 
and positive cases if possible
   - [ ] Add screenshots for manual tests if appropriate
   - [x] Run test locally before making a pull request
   
   ```shell
   ./mvnw -q -pl amoro-format-iceberg -am test
   ./mvnw -q -pl amoro-format-iceberg -am validate
   ```
   
   ## Documentation
   
   - Does this pull request introduce a new feature? no
   - If yes, how is the feature documented? not applicable
   


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