Skip to content

Stabilize flaky embedded tests#19188

Open
Fly-Style wants to merge 1 commit intoapache:masterfrom
Fly-Style:stabilize-flaky-embedded-tests
Open

Stabilize flaky embedded tests#19188
Fly-Style wants to merge 1 commit intoapache:masterfrom
Fly-Style:stabilize-flaky-embedded-tests

Conversation

@Fly-Style
Copy link
Contributor

  • stabilize FaultyClusterTest by ensuring that Druid has ingested rows.
  • increased timeouts on test_run100Tasks_concurrently and test_run25Tasks_*

// Additionally, confirm that rows were placed into Druid.
// Caution: "ingest/rows/output" does not have a 'DATASOURCE' dimension.
indexer.latchableEmitter().waitForEventAggregate(
event -> event.hasMetricName("ingest/rows/output"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't necessary now that #19177 is published, since waitUntilPublishedRecordsAreIngested will check the new metric ingest/rows/published. This one is even better than ingest/rows/output. (ingest/rows/output is emitted when we persist to local disk, ingest/rows/published is emitted when we persist to S3 and add a record to the metadata store, meaning the ingestion is truly done).

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.

2 participants