Skip to content

Commit 544f43c

Browse files
committed
Fix ResultTimeout
1 parent 3f84647 commit 544f43c

File tree

2 files changed

+19
-11
lines changed

2 files changed

+19
-11
lines changed

lib/ResultUtils.h

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,23 @@ inline bool isResultRetryable(Result result) {
3131
return true;
3232
}
3333

34-
static const std::unordered_set<Result> fatalResults{
35-
ResultConnectError, ResultAuthenticationError,
36-
ResultAuthorizationError, ResultInvalidUrl,
37-
ResultInvalidConfiguration, ResultIncompatibleSchema,
38-
ResultTopicNotFound, ResultOperationNotSupported,
39-
ResultNotAllowedError, ResultChecksumError,
40-
ResultCryptoError, ResultConsumerAssignError,
41-
ResultProducerBusy, ResultConsumerBusy,
42-
ResultLookupError, ResultTooManyLookupRequestException};
34+
static const std::unordered_set<Result> fatalResults{ResultConnectError,
35+
ResultTimeout,
36+
ResultAuthenticationError,
37+
ResultAuthorizationError,
38+
ResultInvalidUrl,
39+
ResultInvalidConfiguration,
40+
ResultIncompatibleSchema,
41+
ResultTopicNotFound,
42+
ResultOperationNotSupported,
43+
ResultNotAllowedError,
44+
ResultChecksumError,
45+
ResultCryptoError,
46+
ResultConsumerAssignError,
47+
ResultProducerBusy,
48+
ResultConsumerBusy,
49+
ResultLookupError,
50+
ResultTooManyLookupRequestException};
4351
return fatalResults.find(result) == fatalResults.cend();
4452
}
4553

tests/MockClientImpl.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,11 +80,11 @@ class MockClientImpl : public ClientImpl {
8080
static SyncOpResult wait(const PromisePtr& promise) {
8181
using namespace std::chrono;
8282
auto future = promise->get_future();
83-
auto status = future.wait_for(std::chrono::seconds(3));
83+
auto status = future.wait_for(std::chrono::seconds(10));
8484
if (status == std::future_status::ready) {
8585
return future.get();
8686
} else {
87-
return {ResultTimeout, -1L};
87+
return {ResultUnknownError, -1L};
8888
}
8989
}
9090
};

0 commit comments

Comments
 (0)