Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

integration.testDLQFromCassandra: Fix flakiness, speedup by 4x #168

Merged
merged 2 commits into from
Apr 26, 2017
Merged

integration.testDLQFromCassandra: Fix flakiness, speedup by 4x #168

merged 2 commits into from
Apr 26, 2017

Conversation

venkat1109
Copy link
Contributor

Before:

GOMAXPROCS=2 go-test ./test/integration -testify.m TestDLQWithCassandra

--- PASS: TestDLQWithCassandra (80.67s)
--- PASS: TestNetIntegrationSuiteParallelA (91.41s)

After

--- PASS: TestDLQWithCassandra (22.25s)
--- PASS: TestNetIntegrationSuiteParallelA (33.16s)

@venkat1109 venkat1109 self-assigned this Apr 24, 2017
@venkat1109 venkat1109 requested a review from kirg April 24, 2017 19:42
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0004%) to 68.611% when pulling 37ac60c on integtest into 6442730 on master.

@venkat1109 venkat1109 merged commit 1e9818d into master Apr 26, 2017
@venkat1109 venkat1109 deleted the integtest branch April 26, 2017 23:21
kirg pushed a commit that referenced this pull request Apr 28, 2017
Kafka-for-Cherami integration test (#176)
226fb2c 
* ttt

* integration test for kafka-for-cherami

* integration.testDLQFromCassandra: Fix flakiness, speedup by 4x (#168)

* parallelize

* log kafka errors, if any

* disable flaky tests: TestStoreHostTimerQueueWrite{Then,With}Read (#177)

* retry on kafka publish error

* fix race when updating kafka config (#179)

* ttt

* integration test for kafka-for-cherami

* parallelize

* log kafka errors, if any

* retry on kafka publish error

* ~

* print msg on validation failure

* ~
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kirg kirg kirg approved these changes

Assignees

@venkat1109 venkat1109

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@venkat1109 @coveralls @kirg