-
Notifications
You must be signed in to change notification settings - Fork 19
Implement Config APIs #177
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
base: main
Are you sure you want to change the base?
Conversation
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] |
There was a problem hiding this comment.
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] |
There was a problem hiding this comment.
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] |
There was a problem hiding this comment.
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: [ |
There was a problem hiding this comment.
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 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert renaming this.
This change implements
describeConfigs,alterConfigs, andincrementalAlterConfigson the admin client.on-behalf-of: @SAP [email protected]