aglinxinyuan commented on PR #6083: URL: https://github.com/apache/texera/pull/6083#issuecomment-4881078778
Thanks — done both. `allNames` is now built reflectively (every no-arg `String` accessor named `ENV_*` on the object), so a newly added constant is picked up automatically and inherits the well-formedness and uniqueness checks for free — no hand-maintained mirror to drift. I also swapped `name.trim should not be empty` for `fullyMatch regex "[A-Z][A-Z0-9_]*"`, which rejects the padded-name case (`"STORAGE_JDBC_URL "`) as well as all-whitespace. Pushed in 0a73119. -- 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]
