vogievetsky opened a new pull request #11620: URL: https://github.com/apache/druid/pull/11620
This PR aims to make the lookup view more friendly. Here is the before and after screenshots of the lookup view: ### Before:  ### After:  The notable changes are: - `Poll period` and `Summary` are added as additional info columns - By default the table is ordered on `Lookup name` now - Validation of the form has been updated to make `Poll period` required for extraction lookups as not setting it is guaranteed to be a time bomb for when a new server is spun up and the lookups go out of sync. - Error reporting in forms has been improved (see screenshot below) - The script that extracts SQL functions from the docs has been made more robust (it was not finding the `MV_*` functions before) - Data loader can auto detect more delimiter types New inline error validation:  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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
