Skip to content

[To dev/1.3] [Pipe] Stabilize compression integration test (#18644) - #18650

Open
Caideyipi wants to merge 1 commit into
apache:dev/1.3from
Caideyipi:cp-18644-dev-1.3
Open

Caideyipi wants to merge 1 commit into
apache:dev/1.3from
Caideyipi:cp-18644-dev-1.3

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Description

Backport the applicable compression-test fix from #18644 (2838c35317ff4c0fda0eaec78b141d08d6ee34d0) to dev/1.3.

IoTDBPipeSinkCompressionIT issued six rapid writes to the same time series with now() timestamps. Because timestamps have millisecond precision, multiple writes can receive the same timestamp and be deduplicated, leaving the final row count below eight. This change uses deterministic timestamps 3 through 8 instead.

The upstream commit also updates IoTDBShowReceiversIT. That test and its SHOW RECEIVERS feature are not present on dev/1.3, so that part is intentionally omitted. The compression test also lives under the older autocreate package on this branch; the conflict was resolved at that path.

Verification

  • mvn spotless:check -pl integration-test -P with-integration-tests: passed.
  • git diff origin/dev/1.3..HEAD --check: passed.
  • IoTDBPipeSinkCompressionIT#testCompression1 was attempted locally, but the integration-test module is currently blocked before execution by unrelated existing cross-module dependency errors (missing RepairDataPartitionTableProgressState, Jackson/Gson, and OPC UA classes). The corresponding targeted test passed in [Pipe] Stabilize receiver and compression integration tests #18644.

This PR has:

  • been self-reviewed.
  • modified an existing integration test for the affected path.

Key changed/added classes
  • IoTDBPipeSinkCompressionIT

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.

1 participant