Skip to content

Commit

Permalink
Merge pull request #573 from twmb/misc
Browse files Browse the repository at this point in the history
Misc
  • Loading branch information
twmb authored Sep 21, 2023
2 parents e2f01a2 + 2655e1f commit 33a0fac
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 5 deletions.
8 changes: 7 additions & 1 deletion generate/definitions/50_describe_user_scram_credentials
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,9 @@ DescribeUserSCRAMCredentialsRequest => key 50, max version 0, flexible v0+
// DescribeUserSCRAMCredentialsRequest.
DescribeUserSCRAMCredentialsResponse =>
ThrottleMillis
// The request-level error code. This is 0 except for user or infra issues.
// The request-level error code. This is 0 except for auth or infra issues.
//
// CLUSTER_AUTHORIZATION_FAILED if you do not have DESCRIBE on CLUSTER.
ErrorCode: int16
// The request-level error message, if any.
ErrorMessage: nullable-string
Expand All @@ -24,6 +26,10 @@ DescribeUserSCRAMCredentialsResponse =>
// The name this result corresponds to.
User: string
// The user-level error code.
//
// RESOURCE_NOT_FOUND if the user does not exist or has no credentials.
//
// DUPLICATE_RESOURCE if the user is requested twice+.
ErrorCode: int16
// The user-level error message, if any.
ErrorMessage: nullable-string
Expand Down
12 changes: 9 additions & 3 deletions pkg/kadm/metadata.go
Original file line number Diff line number Diff line change
Expand Up @@ -372,7 +372,9 @@ func (l ListedOffsets) KOffsets() map[string]map[int32]kgo.Offset {

// ListStartOffsets returns the start (oldest) offsets for each partition in
// each requested topic. In Kafka terms, this returns the log start offset. If
// no topics are specified, all topics are listed.
// no topics are specified, all topics are listed. If a requested topic does
// not exist, no offsets for it are listed and it is not present in the
// response.
//
// If any topics being listed do not exist, a special -1 partition is added
// to the response with the expected error code kerr.UnknownTopicOrPartition.
Expand All @@ -384,7 +386,8 @@ func (cl *Client) ListStartOffsets(ctx context.Context, topics ...string) (Liste

// ListEndOffsets returns the end (newest) offsets for each partition in each
// requested topic. In Kafka terms, this returns high watermarks. If no topics
// are specified, all topics are listed.
// are specified, all topics are listed. If a requested topic does not exist,
// no offsets for it are listed and it is not present in the response.
//
// If any topics being listed do not exist, a special -1 partition is added
// to the response with the expected error code kerr.UnknownTopicOrPartition.
Expand All @@ -399,7 +402,8 @@ func (cl *Client) ListEndOffsets(ctx context.Context, topics ...string) (ListedO
// latest offset. In Kafka terms, committed means the last stable offset, and
// newest means the high watermark. Record offsets in active, uncommitted
// transactions will not be returned. If no topics are specified, all topics
// are listed.
// are listed. If a requested topic does not exist, no offsets for it are
// listed and it is not present in the response.
//
// If any topics being listed do not exist, a special -1 partition is added
// to the response with the expected error code kerr.UnknownTopicOrPartition.
Expand All @@ -414,6 +418,8 @@ func (cl *Client) ListCommittedOffsets(ctx context.Context, topics ...string) (L
// returned from this function also include the timestamp of the offset. If no
// topics are specified, all topics are listed. If a partition has no offsets
// after the requested millisecond, the offset will be the current end offset.
// If a requested topic does not exist, no offsets for it are listed and it is
// not present in the response.
//
// If any topics being listed do not exist, a special -1 partition is added
// to the response with the expected error code kerr.UnknownTopicOrPartition.
Expand Down
8 changes: 7 additions & 1 deletion pkg/kmsg/generated.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 33a0fac

Please sign in to comment.