Skip to content

Commit

Permalink
test(elasticsearch): Revert flush period to 1, smoke tests have no to…
Browse files Browse the repository at this point in the history
…lerance for larger values
  • Loading branch information
david-leifker committed Oct 26, 2022
1 parent dee38bb commit d5568d9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ elasticsearch:
bulkProcessor:
async: ${ES_BULK_ASYNC:true}
requestsLimit: ${ES_BULK_REQUESTS_LIMIT:1000}
flushPeriod: ${ES_BULK_FLUSH_PERIOD:10}
flushPeriod: ${ES_BULK_FLUSH_PERIOD:1}
numRetries: ${ES_BULK_NUM_RETRIES:3}
retryInterval: ${ES_BULK_RETRY_INTERVAL:1}
refreshPolicy: ${ES_BULK_REFRESH_POLICY:NONE}
Expand Down
2 changes: 1 addition & 1 deletion smoke-test/tests/delete/delete_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ def test_setup():
rollback_url = f"{gms_host}/runs?action=rollback"
session.post(rollback_url, data=json.dumps({"runId": ingested_dataset_run_id, "dryRun": False, "hardDelete": True, "safe": False}))

sleep(3)
sleep(5)

assert "browsePaths" not in get_aspects_for_entity(entity_urn=dataset_urn, aspects=["browsePaths"], typed=False)
assert "editableDatasetProperties" not in get_aspects_for_entity(entity_urn=dataset_urn, aspects=["editableDatasetProperties"], typed=False)
Expand Down

0 comments on commit d5568d9

Please sign in to comment.