Skip to content

[#14002] improvement(test): Move Hive callers to @HiveFixture; add ClickHouse fixtures and scope (part 3/6) - #14009

Open
nevzheng wants to merge 11 commits into
apache:mainfrom
nevzheng:claude/13900-tc-05-hive-callers-clickhouse-scope
Open

nevzheng wants to merge 11 commits into
apache:mainfrom
nevzheng:claude/13900-tc-05-hive-callers-clickhouse-scope

Conversation

@nevzheng

@nevzheng nevzheng commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

What changes were proposed in this pull request?

This PR includes the commits of the PRs below it. Review only its own commit: nevzheng/gravitino@claude/13900-tc-04-images-moto-ranger-hive...claude/13900-tc-05-hive-callers-clickhouse-scope

Why are the changes needed?

Part 3 of 6 of #14002.

Fix: #14002

Does this PR introduce any user-facing change?

No. Test code only.

How was this patch tested?

Docker runs of every migrated module; failures seen were reproduced on main too.

Stack: #14103 → #14083 → #14084 → #14085 → #14005 → #14006 → #14052 → #14076 → #14007 → #14008 → #14009 → #14010 → #14011 → #14012 → #14013 → #14079 → #14077 → #14078 → #14086 → #14087 → #14088 → #14089 → #14090 → #14091 → #14094

@nevzheng nevzheng self-assigned this Oct 20, 2026
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch 9 times, most recently from 1f7f13c to b679da4 Compare October 21, 2026 06:19
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch from b679da4 to 658f466 Compare October 21, 2026 07:09
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch from 658f466 to ea46217 Compare October 21, 2026 17:20
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch 4 times, most recently from 7f1791b to 7e94c28 Compare October 21, 2026 19:44
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch 2 times, most recently from 02166cd to 264ba96 Compare October 21, 2026 21:16
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch 9 times, most recently from dc20aaa to 261edfe Compare October 22, 2026 03:28
Design doc for apache#13900 and a rumdl config for design-docs/testing.
…e ContainerSuite lifecycle

- Log every container start with pull, start and readiness times; add test
  log4j configs so the lines show.
- Reset all ContainerSuite state in close(); key Hive by version; make the
  PostgreSQL retry wait; pin StarRocks; drop a dead start method.
- Poll readiness with no initial delay; run cleanDisk() only on real starts.
- idp-basic starts its containers once per run; Ryuk back on, unprivileged.
- Drop the suite-wide closes in Hive2, Flink and Spark ITs.
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch from 261edfe to adbdd76 Compare October 22, 2026 04:29
…est containers (part 1/6)

ContainerSpec, ContainerExtension and @FixtureContainer: a test declares a
container with a fixture annotation and receives it as a parameter or field.
Fixtures for Kafka, LocalStack, RustFS, StarRocks, OceanBase and Doris, with
their callers migrated.
…o, Ranger and Hive (part 2/6)

- TestImages declares every test image in one place.
- BaseContainer subclasses can supply their own container (ready for the
  official Testcontainers modules).
- Moto, Ranger and Hive (version, auth) fixtures.
- A container whose start fails is closed instead of leaking.
…add ClickHouse fixtures and scope (part 3/6)

- client-java, fileset, Hudi, Iceberg, Paimon, Iceberg REST, GVFS, Flink,
  Spark and Trino env tests use @HiveFixture.
- ClickHouse (1 or 3 nodes) and ZooKeeper fixtures.
- scope = RUN | CLASS | METHOD; HadoopUserAuthenticationIT gets a private
  Kerberos Hive.
@nevzheng
nevzheng force-pushed the claude/13900-tc-05-hive-callers-clickhouse-scope branch from adbdd76 to 7186e6f Compare October 22, 2026 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Subtask] Add a JUnit fixture framework for test containers and move every caller to it

1 participant