-
Notifications
You must be signed in to change notification settings - Fork 954
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
Use new config definition #55
Conversation
@ewencp JDBC connector using new config definition. |
|
||
public static final String BATCH_MAX_ROWS_CONFIG = "batch.max.rows"; | ||
private static final String BATCH_MAX_ROWS_DOC = | ||
"Maximum number of rows to include in a single batch when polling for new data. This " | ||
+ "setting can be used to limit the amount of data buffered internally in the connector."; | ||
public static final int BATCH_MAX_ROWS_DEFAULT = 100; | ||
private static final String BATCH_MAX_ROWS_DISPLAY = "Batch Max Rows"; |
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.
How many characters do we have? Perhaps "Maximum number of rows per batch"? or "Rows per batch limit"?
Display should probably be close to human english....
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.
The length of the display name should be close to the length of the typical value for the config. How about "Max rows per batch" ?
…alls to get database metadata in a simple start/stop test.
LGTM. |
No description provided.