doris icon indicating copy to clipboard operation
doris copied to clipboard

[fix](fold_constant) add defer in fold constant to close

Open zhangstar333 opened this issue 2 years ago • 32 comments

Proposed changes

when set this.enableFoldConstantByBe = true; and doing prepare work will return error for some reason, eg: can't find function in BE. so will return directly without call close function. Issue Number: close #xxx

Problem summary

Describe your changes.

Checklist(Required)

  1. Does it affect the original behavior:
    • [ ] Yes
    • [ ] No
    • [ ] I don't know
  2. Has unit tests been added:
    • [ ] Yes
    • [ ] No
    • [ ] No Need
  3. Has document been added or modified:
    • [ ] Yes
    • [ ] No
    • [ ] No Need
  4. Does it need to update dependencies:
    • [ ] Yes
    • [ ] No
  5. Are there any changes that cannot be rolled back:
    • [ ] Yes (If Yes, please explain WHY)
    • [ ] No

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

zhangstar333 avatar Feb 09 '23 08:02 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 09 '23 08:02 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 09 '23 08:02 github-actions[bot]

PR approved by at least one committer and no changes requested.

github-actions[bot] avatar Feb 09 '23 09:02 github-actions[bot]

PR approved by anyone and no changes requested.

github-actions[bot] avatar Feb 09 '23 09:02 github-actions[bot]

TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.08 seconds stream load tsv: 483 seconds loaded 74807831229 Bytes, about 147 MB/s stream load json: 39 seconds loaded 2358488459 Bytes, about 57 MB/s stream load orc: 68 seconds loaded 1101869774 Bytes, about 15 MB/s stream load parquet: 28 seconds loaded 861443392 Bytes, about 29 MB/s https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230217132406_clickbench_pr_99153.html

hello-stephen avatar Feb 09 '23 11:02 hello-stephen

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 10 '23 02:02 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 10 '23 09:02 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 15 '23 08:02 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 15 '23 12:02 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 16 '23 02:02 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 17 '23 03:02 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 21 '23 04:02 github-actions[bot]

run p0

zhangstar333 avatar Feb 21 '23 04:02 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 21 '23 04:02 github-actions[bot]

run p0

zhangstar333 avatar Feb 21 '23 10:02 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 21 '23 10:02 github-actions[bot]

run p0

zhangstar333 avatar Feb 21 '23 14:02 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 21 '23 14:02 github-actions[bot]

run p0

zhangstar333 avatar Feb 22 '23 02:02 zhangstar333

run p0

zhangstar333 avatar Feb 22 '23 03:02 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 22 '23 03:02 github-actions[bot]

run p0

zhangstar333 avatar Feb 22 '23 09:02 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 22 '23 09:02 github-actions[bot]

run p0

zhangstar333 avatar Feb 22 '23 13:02 zhangstar333

run p0

zhangstar333 avatar Feb 28 '23 09:02 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 28 '23 09:02 github-actions[bot]

run p0

zhangstar333 avatar Mar 01 '23 02:03 zhangstar333

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Mar 01 '23 02:03 github-actions[bot]

run p0

zhangstar333 avatar Mar 02 '23 03:03 zhangstar333

run p0

zhangstar333 avatar Mar 02 '23 09:03 zhangstar333