plchy2 commented on PR #60558: URL: https://github.com/apache/doris/pull/60558#issuecomment-3854647320
### What problem was fixed? The session variable `query_timeout` validation currently throws `UnsupportedOperationException` for invalid values. This is inconsistent with other FE validations and results in unclear error handling for invalid user input. Additionally, non-numeric values could cause an unhandled `NumberFormatException`. ### How was it fixed? - Replaced `UnsupportedOperationException` with `AnalysisException` - Added explicit number format validation for `query_timeout` - Improved error messages for invalid values ### Behavior changes **Before** - Invalid `query_timeout` values threw `UnsupportedOperationException` - Non-numeric values could cause unclear runtime exceptions **After** - Invalid values throw `AnalysisException` - Non-numeric input is handled gracefully with clear error messages ### Impact This change improves consistency of session variable validation and provides clearer feedback to users without affecting existing valid configurations. -- 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]
