-
Notifications
You must be signed in to change notification settings - Fork 6.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update check-large-objects.sh to be language neutral #61130
Conversation
The previous implementation uses `total` keyword to filter out lines that contains the total size of objects under a directory. But when the OS uses other language, it fails to filter out that line, so the script would fail. This implementation changes the script to match only lines that contains object file by their extension. This implementation would be language neutral.
Update check-large-objects.sh to be language neutral
Seems like I need to ask for approval if I am asked to sign a CLA, according to my Company's policy. I need to figure that out. |
This is an automated comment for commit 2661890 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page
Successful checks
|
I see there are two checks failed, because I need 'can be tested' label. But since this PR is very small, I think it would be fine to skip those tests. |
I see that two fuzzy test failed, is there a guideline for debugging fuzzy test? How can we know which query caused the error? Logical error: 'Unexpected return type from equals. Expected Nullable. Got UInt8'. |
Seems like the detailed error is here: Error on processing query 'SELECT s FROM buffer_table1__fuzz_25 PREWHERE x = materialize(1)': Received from localhost:9000. DB::Exception: Illegal types of arguments (Array(Nullable(Float64)), UInt8) of function equals: While processing x = materialize(1). Stack trace:
SELECT s |
I can't reproduce the bug by the following:
Hi Alexey, any insights? |
Hello, shall we merge this by hand? This PR is very trivial. |
The previous implementation uses
total
keyword to filter out lines that contains the total size of objects under a directory. But when the OS uses other language, it fails to filter out that line, so the script would fail.This implementation changes the script to match only lines that contains object file by their extension. This implementation would be language neutral.
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Update check-large-objects.sh to be language neutral
Documentation entry for user-facing changes