yiguolei commented on code in PR #23325:
URL: https://github.com/apache/doris/pull/23325#discussion_r1301398689
##########
be/src/vec/core/field.h:
##########
@@ -523,148 +537,54 @@ class Field {
return get<T>();
}
- bool operator<(const Field& rhs) const {
Review Comment:
这个不要这么改，<=> 对于一个struct，这个r<=>
是跟字段出现的先后顺序有关，我们一些类型，不是简单类型，可能来回定义不一致。而有的时候我们这些比较运算会放入到存储中，此时可能修改或者refactor
后，结果就不对了。
##########
be/src/vec/core/field.h:
##########
@@ -523,148 +537,54 @@ class Field {
return get<T>();
}
- bool operator<(const Field& rhs) const {
Review Comment:
这个不要这么改，<=> 对于一个struct，这个r<=>
是跟字段出现的先后顺序有关，我们一些类型，不是简单类型，可能来回定义不一致。而有的时候我们这些比较运算会放入到存储中，此时可能修改或者refactor
后，结果就不对了。
--
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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
