You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
[Improvement]: Optimize ams configuration to support parsing of time interval and storage related configuration items when both values and units are specified
#3418
Closed
3 tasks done
Jzjsnow opened this issue
Jan 23, 2025
· 0 comments
· Fixed by #3423
I have searched in the issues and found no similar issues.
What would you like to be improved?
Currently, the ams configuration is inconsistent in the type of time interval related configuration items, some of which are of Duration type, such as http-server.session-timeout, and some of which are of Long type, such as refresh-external-catalogs.interval, which may be misused by the user.
In addition, storage-related parameters such as thrift-server.max-message-size only support Long type (default unit is byte). It would be more user-friendly to support user-specified units, as in the Apache Flink configuration.
Search before asking
What would you like to be improved?
Currently, the ams configuration is inconsistent in the type of time interval related configuration items, some of which are of
Duration
type, such ashttp-server.session-timeout
, and some of which are ofLong
type, such asrefresh-external-catalogs.interval
, which may be misused by the user.In addition, storage-related parameters such as
thrift-server.max-message-size
only supportLong
type (default unit is byte). It would be more user-friendly to support user-specified units, as in the Apache Flink configuration.How should we improve?
No response
Are you willing to submit PR?
Subtasks
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: