Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for retrying for message delivery errors #1053

Merged
merged 3 commits into from
Nov 20, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions examples/kafka-hub/docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ services:
KEYSTORE_PASSWORD: "password"
# Maximum number of records returned in a single call to consumer-poll
KAFKA_CONSUMER_MAX_POLL_RECORDS: 50
# The HTTP status codes for which the client should retry
RETRYABLE_STATUS_CODES: "500 502 503"
volumes:
# Kafka client truststore file
- ./_resources/secrets/kafka-client/kafka-client.trustStore.jks:/home/ballerina/resources/brokercerts/client-truststore.jks
Expand Down
3 changes: 3 additions & 0 deletions examples/kafka-hub/hub/Config.toml
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ MESSAGE_DELIVERY_COUNT = 3
# The message delivery timeout
MESSAGE_DELIVERY_TIMEOUT = 10.0

# The HTTP status codes for which the client should retry
MESSAGE_DELIVERY_RETRYABLE_STATUS_CODES = [500, 502, 503]

# The Oauth2 authorization related configurations
[kafkaHub.config.OAUTH2_CONFIG]
issuer = "https://localhost:9443/oauth2/token"
Expand Down
13 changes: 13 additions & 0 deletions examples/kafka-hub/hub/modules/config/configurations.bal
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,11 @@ public configurable int MESSAGE_DELIVERY_COUNT = 3;
# The message delivery timeout
public configurable decimal MESSAGE_DELIVERY_TIMEOUT = 10;

# The HTTP status codes for which the client should retry
public configurable int[] MESSAGE_DELIVERY_RETRYABLE_STATUS_CODES = [500, 502, 503];

public final readonly & int[] RETRYABLE_STATUS_CODES = check getRetryableStatusCodes(MESSAGE_DELIVERY_RETRYABLE_STATUS_CODES).cloneReadOnly();

# The Oauth2 authorization related configurations
public configurable types:OAuth2Config OAUTH2_CONFIG = ?;

Expand All @@ -76,3 +81,11 @@ public final string WEBSUB_EVENTS_CONSUMER_GROUP = os:getEnv("WEBSUB_EVENTS_CONS
isolated function constructSystemConsumerGroup() returns string {
return string `websub-events-receiver-${SERVER_IDENTIFIER}-${util:generateRandomString()}`;
}

isolated function getRetryableStatusCodes(int[] configuredCodes) returns int[]|error {
if os:getEnv("RETRYABLE_STATUS_CODES") is "" {
return configuredCodes;
}
string[] statusCodes = re ` `.split(os:getEnv("RETRYABLE_STATUS_CODES"));
ayeshLK marked this conversation as resolved.
Show resolved Hide resolved
return statusCodes.'map(i => check int:fromString(i));
}
3 changes: 2 additions & 1 deletion examples/kafka-hub/hub/websub_subscribers.bal
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,8 @@ isolated function pollForNewUpdates(string subscriberId, websubhub:VerifiedSubsc
interval: config:MESSAGE_DELIVERY_RETRY_INTERVAL,
count: config:MESSAGE_DELIVERY_COUNT,
backOffFactor: 2.0,
maxWaitInterval: 20
maxWaitInterval: 20,
statusCodes: config:RETRYABLE_STATUS_CODES
},
timeout: config:MESSAGE_DELIVERY_TIMEOUT
});
Expand Down
Loading