zanarellidev commented on PR #7639: URL: https://github.com/apache/kyuubi/pull/7639#issuecomment-5360870232
Done, pushed e01fbd846130fc1a6d9147104f3cd1cba9960aa4. Reverted both doc strings to the pre-PR wording and regenerated docs/configuration/settings.md via dev/gen/gen_all_config_docs.sh. That regen actually caught a real bug: the new default `(?i)secret|password|token|access[.]key` is the first regex default with a literal `|` in it, and AllKyuubiConfiguration's table builder joins cells with `|` and doesn't escape it, so the row split into bogus columns. Fixed that too (escaping the Default column only, since the Meaning column already leans on backtick code spans to protect any `|` it has). Added the migration-guide.md entry as well. -- 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]
