yiguolei commented on code in PR #52104:
URL: https://github.com/apache/doris/pull/52104#discussion_r2165537984
##########
be/src/vec/data_types/serde/data_type_serde.h:
##########
@@ -329,6 +329,15 @@ class DataTypeSerDe {
return serialize_one_cell_to_json(column, row_num, bw, options);
}
+ virtual Status serialize_column_to_jsonb(const IColumn& from_column,
int64_t row_num,
+ JsonbWriter& writer) const {
+ return Status::NotSupported(
+ "DataTypeSerDe::serialize_column_to_jsonb is not supported for
this data type");
Review Comment:
这里直接输出 data type的名字，这样用户直观的了解，否则比如嵌套array的时候，都不知道哪一层出错了
--
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]
