Danny Chan

Results 483 comments of Danny Chan

You can take a look at the readme: https://github.com/apache/hudi/tree/master/hudi-integ-test

You can have a try of our new NB-CC concurrency mode, here is what it looks like with Flink options: ```sql -- NB-CC demo CREATE TABLE sourceT ( uuid varchar(20),...

In SQL, a tricky solution is when pasing the SQL, if a column defines a default value, the parser would translate and wrapper it as a function with a null...

> Overall it looks solid to me. Only concern is that this code is very critical and complex, do we have enough tests to ensure the correctness? We have all...

Just rollback the greater commits with Hudi CLI and restart the job.

See the log report: > Caused by: org.apache.hudi.exception.HoodieRollbackException: Found commits after time :20231214220739609, please rollback greater commits first

I see many test failures the input stream of log reader can not be closed: https://github.com/apache/hudi/actions/runs/7228545731/job/19698162762?pr=10340#step:6:7650

Yeah,it is already cherry-picked into 0.14.1 release, just resolve the conflicts and we are good to merge.

Just to note: we are migrating our incremental query semantics to base on completion time: https://github.com/apache/hudi/pull/10255, this change might be relevent.