-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[Feature](Prepared Statement) fix and enable enable_server_side_prepared_statement by default #36581 #36818
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
Conversation
|
run buildall |
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
4aa3b69 to
20366a8
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
| if (item == "nereids") { | ||
| statement.execute("set enable_nereids_dml = true;"); | ||
| statement.execute("set enable_nereids_planner=true;"); | ||
| <<<<<<< HEAD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
…red_statement by default (apache#36581) 1. fix insert with label should not go server side prepared 2. add max_prepared_stmt_count sessoin var to avoid too many prepared statements
|
run buildall |
|
run buildall |
|
run buildall |
picked from #36581