Skip to content
This repository has been archived by the owner on Nov 20, 2020. It is now read-only.

Error messages in the consumer #150

Open
dservgun opened this issue Jun 9, 2017 · 0 comments
Open

Error messages in the consumer #150

dservgun opened this issue Jun 9, 2017 · 0 comments

Comments

@dservgun
Copy link

dservgun commented Jun 9, 2017

0001-Added-error-codes-for-consumer.patch.txt
Should we add the following error codes in the consumer? The generic message reads "unsupported commit offset error_code=%i" ec)) if we accept this change the generic message may need to reflect a kafka error?

                   | ErrorCode.NoError -> None
-                  | ErrorCode.IllegalGenerationCode | ErrorCode.UnknownMemberIdCode 
+                  | ErrorCode.BrokerNotAvailable | ErrorCode.ClusterAuthorizationFailedCode
+                  | ErrorCode.GroupAuthorizationFailedCode | ErrorCode.GroupCoordinatorNotAvailableCode
+                  | ErrorCode.GroupLoadInProgressCode | ErrorCode.InconsistentGroupProtocolCode
+                  | ErrorCode.InvalidCommitOffsetSizeCode | ErrorCode.InvalidGroupIdCode
+                  | ErrorCode.InvalidMessage | ErrorCode.InvalidMessageSize
+                  | ErrorCode.InvalidRequiredAcksCode | ErrorCode.InvalidSessionTimeoutCode
+                  | ErrorCode.InvalidTopicCode | ErrorCode.LeaderNotAvailable
+                  | ErrorCode.MessageSizeTooLarge | ErrorCode.NotEnoughReplicasAfterAppendCode
+                  | ErrorCode.NotLeaderForPartition | ErrorCode.RecordListTooLargeCode
+                  | ErrorCode.ReplicaNotAvailable | ErrorCode.RequestTimedOut
+                  | ErrorCode.StaleControllerEpochCode | ErrorCode.TopicAuthorizationFailedCode
+                  | ErrorCode.UnknownTopicOrPartition
+                  | ErrorCode.IllegalGenerationCode | ErrorCode.UnknownMemberIdCode
                   | ErrorCode.RebalanceInProgressCode | ErrorCode.NotCoordinatorForGroupCode -> Some (p,ec)
                   | _ -> failwithf "general server error_code=%i" ec))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant