voonhous commented on PR #19161: URL: https://github.com/apache/hudi/pull/19161#issuecomment-5291697892
Also addressed the remaining nits in c7c6058: collapsed the three `copy_to_temp_view` query-type tests into one MOR test (the RO arm now asserts the pre-update price instead of a count that snapshot would also satisfy), replaced the `sync_validate` contains-checks with exact-message assertions and dropped the duplicated mandatory-args test, extended `repair_clustering_plan` with a `NOT_FOUND_IN_PLAN` case (a file outside the plan is never deleted, even with `need_delete => true`) and a destructive `allow_empty_plan` run asserting the rewritten instant carries an empty plan, tightened the `show_clean_plans` unknown-column assertion, and extracted shared create-table helpers. Net -89 lines with two new coverage points. -- 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]
