github-actions[bot] commented on code in PR #63744:
URL: https://github.com/apache/doris/pull/63744#discussion_r3341970345
##########
fe/fe-common/src/main/java/org/apache/doris/common/Config.java:
##########
@@ -2009,13 +1944,6 @@ public class Config extends ConfigBase {
@ConfField(mutable = true)
public static boolean enable_decimal_conversion = true;
Review Comment:
Removing this config leaves checked-in pytest suites broken. The
`pytest/sys/test_sys_array/*` tests still call
`admin_show_config('enable_array_type')`, assert that one row is returned, and
then call `set_frontend_config('enable_array_type', 'true')`; after this
deletion `SHOW CONFIG` returns no row and `SET FRONTEND CONFIG` reports that
the config does not exist. The same problem exists for
`enable_concurrent_update` in `pytest/sys/test_sys_update_basic.py`. Please
update or remove those test references together with the config deletion so the
repository test suite stays consistent.
--
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]
