vogievetsky opened a new pull request #11228: URL: https://github.com/apache/druid/pull/11228
This PR fixes a bug in the data loader flow when connecting Kinesis. The issue is that the `Endpoint` field looks like it is filled in with the default value but it also is `required` so it requires input. The issue is fixed as a one line change in `ingestion-spec.tsx` per the docs the `endpoint` field is not required. In addition to fixing this issue: - AutoForm has been upgraded so that `defaultValue` is ignored when `required` is evaluated to true - Tests have been greatly expanded - I scanned the code for more instances of this issue and found one in the `TIMESTAMP_SPEC_FIELDS` - Added the ability to Alt+Shift+Click a tile in the ingestion screen to go to its connection step even if the extension is not loaded. This PR has: - [x] been self-reviewed. - [x] been tested in a test Druid cluster. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
