-
Notifications
You must be signed in to change notification settings - Fork 19
Add Apache Doris #73
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
Add Apache Doris #73
Conversation
@rschu1ze Hi, I updated sql and got the new results with accurate machine( |
@BiteTheDDDDt Thanks for updating the results. Perhaps you also like to add measurements on your To double-check, I re-ran the script locally and got this:
As a result, =all result files (e.g. "doris/_m6i.8xlarge_bluesky_100m_default.results_runtime") were empty. I suppose the error is this:
What does that mean exactly and how to fix the error? Also, doris/run_queries.sh contains this: mysql -P 9030 -h 127.0.0.1 -u root $DB_NAME -e "set global parallel_pipeline_task_num=32;"
mysql -P 9030 -h 127.0.0.1 -u root $DB_NAME -e "set global enable_parallel_scan=false;" All databases in JSONBench are ideally run with default settings. If that is not possible, kindly document what the special configuration does and why it is necessary to set. Thanks! |
Sorry to ask again, when I run the script in my virgin Ubuntu 24.04 environment, I am getting error
and all result files are empty. How can this be fixed? |
Seems doris backend not ready, can you check |
@rschu1ze Hi, did you get a chance to check the log? If there's still any problem, feel free to share the output — I’d be happy to help debug further. thanks again for your time and help reviewing this PR! |
The BE logs were empty and the reason was that the BE node didn't start. When I started it manually, it complained (like above) about
I then commented that check in start_be.sh out. This worked finally. Just started the full test suite for reproduction ... |
Hi, I noticed that you've already uploaded your test results. Is there anything else I need to do? If so, please let me know. |
@BiteTheDDDDt Turns out there is now a number of submissions for JSONBench (this PR, but also others) where the structure of the original JSON document is not retained (i.e. flattening, see here). This is fine, however to allow users to distinguish both cases, we'll need a button for the JSONBench UI. I asked a colleague to take a look, then I am happy to merge. Thanks for this PR! |
Following up, the same consideration applies here as in #77. Made the corresponding adjustments (removed the materialized results). |
No description provided.