sql-ds-cache icon indicating copy to clipboard operation
sql-ds-cache copied to clipboard

Ape base

Open yspMing opened this issue 4 years ago • 3 comments

What changes were proposed in this pull request?

(Please fill in changes proposed in this fix)

How was this patch tested?

(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

yspMing avatar Jan 07 '22 08:01 yspMing

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues? https://github.com/oap-project/sql-ds-cache/issues

Then could you also rename pull request title and commit log in the following format?

[SQL-DS-CACHE-${ISSUES_ID}] ${detailed message}

See also:

github-actions[bot] avatar Jan 07 '22 08:01 github-actions[bot]

Thanks a lot for Kunshang’s review.

I commit a new C++ file instead of adding function to the existing expression so that the basic version wouldn’t be polluted. Yes they can be merged and become more clear.

And I’ll look for and the other issues you mentioned and repair these.

yspMing avatar Jan 10 '22 07:01 yspMing

CI failed at code style check stage. Please fix it.

jikunshang avatar Jan 18 '22 08:01 jikunshang