Attila Bukor has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15629 )

Change subject: [postgres] Deflake MiniPostgres tests
......................................................................

[postgres] Deflake MiniPostgres tests

Tests depending on MiniPostgres were flaky as in some cases Postgres
takes longer than expected to start. This commit introduces checking if
psql can connect to MiniPostgres as part of start-up and retries every
100ms for 5 seconds.

Ran MiniRangerTest 1000 times on dist-test with no failures with this
fix.

Change-Id: I5106b3e2aeb9dabad9a7ee0b17059c1df2042bb4
Reviewed-on: http://gerrit.cloudera.org:8080/15629
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
---
M src/kudu/postgres/mini_postgres.cc
M src/kudu/postgres/mini_postgres.h
2 files changed, 28 insertions(+), 3 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Adar Dembo: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/15629
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5106b3e2aeb9dabad9a7ee0b17059c1df2042bb4
Gerrit-Change-Number: 15629
Gerrit-PatchSet: 11
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)

Reply via email to