[Improvement]: Add non-partitioned table test coverage for mixed-hive rewrite and overwrite operations

[xxubai]

Search before asking

• I have searched in the issues and found no similar issues.

What would you like to be improved?

The mixed-hive test suite already covers several rewrite and overwrite scenarios, but multiple test methods still contain the same TODO: add cases for tables without partition spec.

Examples can be found in:

• amoro-format-mixed/amoro-mixed-hive/src/test/java/org/apache/amoro/hive/op/TestRewriteFiles.java
• amoro-format-mixed/amoro-mixed-hive/src/test/java/org/apache/amoro/hive/op/TestOverwriteFiles.java

Currently these cases are guarded by Assume.assumeTrue(isPartitionedTable()), so behavior for non-partitioned tables is not covered well enough.

How should we improve?

We should add non-partitioned table test coverage for mixed-hive rewrite and overwrite operations.

A contributor can start by:

• identifying the existing partitioned-only test cases in TestRewriteFiles and TestOverwriteFiles
• adding equivalent or adapted cases for tables without partition spec
• verifying whether non-partitioned tables should succeed or throw the expected exception in each scenario
• keeping the new tests independent and easy to understand

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Subtasks

• Review the existing partitioned-table cases in TestRewriteFiles
• Review the existing partitioned-table cases in TestOverwriteFiles
• Add non-partitioned table coverage for the missing scenarios
• Run the relevant mixed-hive tests to validate the expected behavior

Code of Conduct

• I agree to follow this project's Code of Conduct

[Takch02]

Hi, I would like to work on this issue as my first code contribution! Since I am currently studying the codebase and this is my first time writing tests for this project, it might take a week or two. Could you assign it to me?

[xxubai]

@Takch02 Sure, go ahead

[JeonDaehong]

@xxubai Hi, there has been no PR for this issue for over 2 weeks. Would it be okay if I work on this and submit a PR?

[Takch02]

@xxubai Hi, there has been no PR for this issue for over 2 weeks. Would it be okay if I work on this and submit a PR?

This is a PR I have been working on for the past two weeks. I would appreciate it if you could give me a chance.

[JeonDaehong]

Ah, I see you were working haha.
Sorry about that 😅