Qing
Qing
Hi @libenchao I finally got around and implemented your suggestion. Do you mind to take a look again? I didn't implement additional operators like IN and BETWEEN because I am...
@flinkbot run azure
@flinkbot run azure
Hi @libenchao , thank you very much for your review 👍 I've addressed all of your concern. On the TablePlanTest, do you mind to check if that's how it supposed...
> One small tip, do not rebase/force-pushing before the reviewer asks you because this will make the reviewer hard to do incremental review. Sorry, my bad, I was advised to...
@flinkbot run azure
@flinkbot run azure
@flinkbot run azure
@libenchao > Thanks for the update. Please do not use 'squash' or 'force-push' unless you must or the reviewer asks you. (I go through all the codes again, and left...
Hi @libenchao , this is the new commit I added to address your comment. https://github.com/apache/flink/pull/20140/commits/42078a44864b74e66a64fa69b8b46cf717be279d I also added support IS NULL and IS NOT NULL as these 2 are quite...