DongLiang-0 commented on code in PR #25364:
URL: https://github.com/apache/doris/pull/25364#discussion_r1366522458
##########
regression-test/suites/external_table_p0/paimon/test_paimon_catalog.groovy:
##########
@@ -72,10 +72,14 @@ suite("test_paimon_catalog",
"p0,external,doris,external_docker,external_docker_
def c15 = """select * from all_table where c15='a';"""
def c16 = """select * from all_table where c16=true;"""
def c18 = """select * from all_table where c18='2023-08-13
09:32:38.53';"""
- def c19 = """select * from auto_bucket;"""
+ def c19 = """select * from auto_bucket order by user_id;"""
def c20 = """select * from auto_bucket where dt="b";"""
def c21 = """select * from auto_bucket where dt="b" and hh="c";"""
def c22 = """select * from auto_bucket where dt="d";"""
+ def c23 = """select * from complex_tab order by c1;"""
+ def c24 = """select * from complex_tab where c1=1;"""
Review Comment:
Thanks for your suggestion, I have changed it
##########
fe/be-java-extensions/paimon-scanner/src/main/java/org/apache/doris/paimon/PaimonColumnValue.java:
##########
@@ -32,12 +35,18 @@
public class PaimonColumnValue implements ColumnValue {
private int idx;
- private InternalRow record;
- ColumnType dorisType;
+ private DataGetters record;
Review Comment:
When dealing with `unpackArray` and `unpackMap`, since
`org.apache.paimon.data.InternalArray` extends
`org.apache.paimon.data.DataGetters`, if do not change the `InternalRow` to
`DataGetters`, the force transfer will fail.
--
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
