-
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?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -83,20 +83,39 @@ export const ConfigSources = { | |
| DYNAMIC_DEFAULT_BROKER_CONFIG: 3, | ||
| STATIC_BROKER_CONFIG: 4, | ||
| DEFAULT_CONFIG: 5, | ||
| DYNAMIC_BROKER_LOGGER_CONFIG: 6 | ||
| DYNAMIC_BROKER_LOGGER_CONFIG: 6, | ||
| CLIENT_METRICS_CONFIG: 7, | ||
| GROUP_CONFIG: 8 | ||
| } as const | ||
| export type ConfigSource = keyof typeof ConfigSources | ||
| export type ConfigSource = (typeof ConfigSources)[keyof typeof ConfigSources] | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Revert this. |
||
|
|
||
| export const ConfigTypes = { | ||
| export const ConfigResources = { | ||
| UNKNOWN: 0, | ||
| TOPIC: 2, | ||
| BROKER: 4, | ||
| BROKER_LOGGER: 8 | ||
| BROKER_LOGGER: 8, | ||
| CLIENT_METRICS: 16, | ||
| GROUP: 32 | ||
| } as const | ||
| export type ConfigResource = (typeof ConfigResources)[keyof typeof ConfigResources] | ||
|
|
||
| export const ConfigTypes = { | ||
| UNKNOWN: 0, | ||
| BOOLEAN: 1, | ||
| STRING: 2, | ||
| INT: 3, | ||
| SHORT: 4, | ||
| LONG: 5, | ||
| DOUBLE: 6, | ||
| LIST: 7, | ||
| CLASS: 8, | ||
| PASSWORD: 9 | ||
| } as const | ||
| export type ConfigType = keyof typeof ConfigTypes | ||
| export type ConfigType = (typeof ConfigTypes)[keyof typeof ConfigTypes] | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. export type ConfigType = keyof typeof ConfigTypes |
||
|
|
||
| export const IncrementalAlterConfigTypes = { SET: 0, DELETE: 1, APPEND: 2, SUBTRACT: 3 } | ||
| 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] | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Revert this |
||
|
|
||
| // ./admin/*-client-quotas.ts | ||
| export const ClientQuotaMatchTypes = { EXACT: 0, DEFAULT: 1, ANY: 2 } as const | ||
|
|
||
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.