Skip to content

Conversation

@lp247
Copy link
Contributor

@lp247 lp247 commented Nov 25, 2025

This change implements describeConfigs, alterConfigs, and incrementalAlterConfigs on the admin client.

on-behalf-of: @SAP [email protected]

This change implements `describeConfigs`, `alterConfigs`, and
`incrementalAlterConfigs` on the admin client.

on-behalf-of: @SAP [email protected]
GROUP_CONFIG: 8
} as const
export type ConfigSource = keyof typeof ConfigSources
export type ConfigSource = (typeof ConfigSources)[keyof typeof ConfigSources]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert this.

PASSWORD: 9
} as const
export type ConfigType = keyof typeof ConfigTypes
export type ConfigType = (typeof ConfigTypes)[keyof typeof ConfigTypes]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

export type ConfigType = keyof typeof ConfigTypes

export type IncrementalAlterConfigType = keyof typeof IncrementalAlterConfigTypes
export const IncrementalAlterConfigOperationTypes = { SET: 0, DELETE: 1, APPEND: 2, SUBTRACT: 3 } as const
export type IncrementalAlterConfigOperationType =
(typeof IncrementalAlterConfigOperationTypes)[keyof typeof IncrementalAlterConfigOperationTypes]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert this

name: { type: 'string', minLength: 1 },
configOperation: {
type: 'number',
enum: [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace with allowedIncrementalAlterConfigOperationTypes as done elsewhere.

type IncrementalAlterConfigOperationType
} from '../enumerations.ts'

export interface IncrementalAlterConfigsRequestConfig {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert renaming this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants