Skip to content

Fixes for completion events post-dispatching#14788

Merged
tdcmeehan merged 1 commit intoprestodb:masterfrom
tdcmeehan:event2
Jul 13, 2020
Merged

Fixes for completion events post-dispatching#14788
tdcmeehan merged 1 commit intoprestodb:masterfrom
tdcmeehan:event2

Conversation

@tdcmeehan
Copy link
Contributor

@tdcmeehan tdcmeehan commented Jul 6, 2020

Improvement on #14784

  • Removes a potential duplicate completion event
  • Additional fix for certain DDL and pre-execution SQL execution problems where completion event may not be logged

@tdcmeehan tdcmeehan marked this pull request as draft July 7, 2020 04:35
@tdcmeehan tdcmeehan marked this pull request as ready for review July 7, 2020 16:02
@tdcmeehan tdcmeehan marked this pull request as draft July 7, 2020 16:21
@tdcmeehan tdcmeehan changed the title Move query completion events to dispatcher Fixes for completion events post-dispatching Jul 8, 2020
@tdcmeehan tdcmeehan marked this pull request as ready for review July 8, 2020 20:50
@tdcmeehan tdcmeehan marked this pull request as draft July 9, 2020 01:25
@tdcmeehan tdcmeehan requested a review from highker July 10, 2020 16:31
@tdcmeehan tdcmeehan marked this pull request as ready for review July 10, 2020 17:25
@tdcmeehan tdcmeehan requested a review from arhimondr July 13, 2020 19:43
@tdcmeehan tdcmeehan merged commit 20a1726 into prestodb:master Jul 13, 2020
@tdcmeehan tdcmeehan deleted the event2 branch July 13, 2020 23:07
@caithagoras caithagoras mentioned this pull request Jul 28, 2020
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants