Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Emit events when assembling insert, update and delete statements #56

Merged
merged 1 commit into from
Jun 10, 2022
Merged

Emit events when assembling insert, update and delete statements #56

merged 1 commit into from
Jun 10, 2022

Conversation

yhabteab
Copy link
Member

To be able to subscribe to those events, like for select statments
and e.g transform binary data to hex for the pgsql adapter

@cla-bot cla-bot bot added the cla/signed label May 19, 2022
@yhabteab yhabteab requested review from lippserd and nilmerg May 19, 2022 13:40
@yhabteab yhabteab force-pushed the introduce-events-for-insert-update-and-delete-statements branch 2 times, most recently from d76b237 to c37ce24 Compare May 19, 2022 14:06
@yhabteab yhabteab mentioned this pull request May 20, 2022
Open
@yhabteab yhabteab force-pushed the introduce-events-for-insert-update-and-delete-statements branch 2 times, most recently from d2565e7 to e740d91 Compare June 7, 2022 14:43
@yhabteab yhabteab requested review from lippserd and nilmerg and removed request for lippserd and nilmerg June 7, 2022 14:45
nilmerg
nilmerg requested changes Jun 10, 2022
View reviewed changes
@nilmerg nilmerg added this to the v0.5.0 milestone Jun 10, 2022
@nilmerg nilmerg added the enhancement New feature or request label Jun 10, 2022
@yhabteab
Emit events when assembling insert, update and delete statements
1ae0e15 
To be able to subscribe to those events, like for select statments
and e.g transform binary data to hex for the pgsql adapter
@yhabteab yhabteab force-pushed the introduce-events-for-insert-update-and-delete-statements branch from e740d91 to 1ae0e15 Compare June 10, 2022 13:40
@yhabteab yhabteab requested a review from nilmerg June 10, 2022 13:40
@nilmerg nilmerg merged commit 110d747 into master Jun 10, 2022
@nilmerg nilmerg deleted the introduce-events-for-insert-update-and-delete-statements branch June 10, 2022 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nilmerg nilmerg nilmerg approved these changes

@lippserd lippserd Awaiting requested review from lippserd

Assignees

@yhabteab yhabteab

Labels
cla/signed enhancement New feature or request
Projects
None yet
Milestone
v0.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@yhabteab @nilmerg