Skip to content

Notify a connector about end of planning#14828

Closed
kokosing wants to merge 8 commits intotrinodb:masterfrom
kokosing:origin/master/156_metadata_finished
Closed

Notify a connector about end of planning#14828
kokosing wants to merge 8 commits intotrinodb:masterfrom
kokosing:origin/master/156_metadata_finished

Conversation

@kokosing
Copy link
Copy Markdown
Member

Notify a connector about end of planning

That way it is possible to close reused JDBC connection faster.

That way tests are more predicable and it is guaranteed that data
processing will be executed in different "node" than metadata queries.
That way it is possible to inject a different way to provision
ConnectionFactory in PostgreSql connector.
@cla-bot cla-bot bot added the cla-signed label Oct 29, 2022
@kokosing kokosing marked this pull request as draft October 29, 2022 21:57
@kokosing
Copy link
Copy Markdown
Member Author

Based on #14653

Statements executed during commitCreateTable are going to change the
state of the remote data base so auto commit mode is expected.
QueryEventListener will be notified when query is started or finished.
That way it is possible to close reused JDBC connection faster.
@kokosing kokosing force-pushed the origin/master/156_metadata_finished branch from 644591a to eb1b472 Compare October 30, 2022 19:46
@kokosing kokosing closed this Feb 20, 2023
@kokosing kokosing deleted the origin/master/156_metadata_finished branch February 20, 2023 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant