-
Notifications
You must be signed in to change notification settings - Fork 235
Adding Arc results for c6a.4xlarge #661
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
Open
xe-nvdk
wants to merge
16
commits into
ClickHouse:main
Choose a base branch
from
Basekick-Labs:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+648
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The Arc API query endpoint has been updated to include the /api/v1 prefix. Old: /query/arrow New: /api/v1/query/arrow This matches the current Arc API structure. 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
…endpoints and added in the logs info about the storage used
…eflected in the script
|
I will add more systems over the week. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey team. Nice sunday eh! Thank you to @alexey-milovidov to mark that the filesystem cache was enabled and that was originating not fair comparison. I want to clarify that wasn't on purpose or with the intention of cheat, and for that reason, we are posting the results for c6a.4xlarge again and yes, cold run is triple that the original, however, we are still happy cause we are perfoming betters than other systems in the category.
I fixed the script to point to the right endpoints, and also added a check for the storage, so, if you have aws-cli installed you can see what type of storage is running, also this is for eliminate any doubts regarding the storage used.
Have a look, and let me know if you have any questions.
Thank you