-
Notifications
You must be signed in to change notification settings - Fork 506
Internal Query Execution Status (Issue #1555) #1561
Conversation
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Codecov Report
@@ Coverage Diff @@
## master #1561 +/- ##
==========================================
+ Coverage 81.97% 81.99% +0.01%
==========================================
Files 735 736 +1
Lines 51698 51726 +28
==========================================
+ Hits 42379 42412 +33
+ Misses 9319 9314 -5
Continue to review full report at Codecov.
|
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
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.
Looks great. Thanks, @17zhangw!
Performance Boost!Nice job! This PR has increased the throughput of the system. Could not find any performance results to compare for this commit. |
Performance Boost!Nice job! This PR has increased the throughput of the system. Could not find any performance results to compare for this commit. |
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Minor Decrease in PerformanceBe warned: this PR may have decreased the throughput of the system slightly.
|
Description
PR resolves issue #1555. The PR augments
TaskDDL
with a future that can be used by the caller to identify whether or not the DDL query has succeeded or failed (along with obtaining an error message associated with the failure). PR also performs the following:util::QueryExecUtil
now tracks a "most-recent" error that can be obtained.util::QueryExecUtil::ExecuteQuery
returns false now if the transaction must abort due to execution