Skip to content

Conversation

@ermyar
Copy link
Collaborator

@ermyar ermyar commented Oct 30, 2025

Removed deprecated pool's methods. Related tests are updated (deleted or refactored). Also removed mentioned methods in interfaces declarations.

Fixes #478

I didn't forget about (remove if it is not applicable):

@ermyar ermyar force-pushed the ermyar/gh-478-removing-deprecated-pool-methods branch from 030b998 to e02f318 Compare October 30, 2025 13:33
@ermyar ermyar requested review from bigbes and oleg-jukovec October 30, 2025 13:45
Copy link
Collaborator

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it still draft? Please, make it ready for review when you'll be ready.

@ermyar ermyar marked this pull request as ready for review October 31, 2025 16:59
@ermyar ermyar requested a review from oleg-jukovec November 1, 2025 00:00
Copy link
Collaborator

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but please add a note about the change into the migration guide:

https://github.com/tarantool/go-tarantool/blob/master/MIGRATION.md?plain=1#L9

@ermyar ermyar force-pushed the ermyar/gh-478-removing-deprecated-pool-methods branch from e02f318 to 0839bd4 Compare November 1, 2025 23:12
@ermyar ermyar requested a review from oleg-jukovec November 2, 2025 09:36
@oleg-jukovec
Copy link
Collaborator

LGTM, but please add a note about the change into the migration guide:

https://github.com/tarantool/go-tarantool/blob/master/MIGRATION.md?plain=1#L9

Still relevant.

Copy link
Collaborator

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, add a note into the MIGRATION.md about the change.

@ermyar ermyar force-pushed the ermyar/gh-478-removing-deprecated-pool-methods branch from 0839bd4 to 021143a Compare November 2, 2025 17:32
@ermyar ermyar requested a review from oleg-jukovec November 2, 2025 17:44
@ermyar ermyar force-pushed the ermyar/gh-478-removing-deprecated-pool-methods branch from 021143a to 5ed5af6 Compare November 2, 2025 20:17
Removed deprecated pool's methods. Related tests are updated
(deleted or refactored). Also removed mentioned methods in
interfaces declarations.

Fixes #478
@ermyar ermyar force-pushed the ermyar/gh-478-removing-deprecated-pool-methods branch from 5ed5af6 to 1e2e17d Compare November 2, 2025 20:19
Copy link
Collaborator

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the patch!

@oleg-jukovec oleg-jukovec merged commit c972ce4 into master Nov 3, 2025
42 of 43 checks passed
@oleg-jukovec oleg-jukovec deleted the ermyar/gh-478-removing-deprecated-pool-methods branch November 3, 2025 10:27
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.

v3: remove all deprecated pooler methods and their implementations

4 participants