Skip to content

Remove deprecated table.rollback()#4777

Closed
ajantha-bhat wants to merge 1 commit intoapache:masterfrom
ajantha-bhat:dep2
Closed

Remove deprecated table.rollback()#4777
ajantha-bhat wants to merge 1 commit intoapache:masterfrom
ajantha-bhat:dep2

Conversation

@ajantha-bhat
Copy link
Copy Markdown
Member

@ajantha-bhat ajantha-bhat commented May 16, 2022

Remove deprecated table.rollback() and RollbackToSnapshot

TODO: update testcases for table.rollback()

@ajantha-bhat ajantha-bhat marked this pull request as draft May 16, 2022 04:37
@kbendick
Copy link
Copy Markdown
Contributor

How long have we had this deprecated @ajantha-bhat? Are you aware of any compatibility guarantees or any timeline we might be bound to before fully removnig this?

It looks as though this has been deprecated for a while, as the old interface is simply written in terms of the new one. But we don't want to break compatibility relatively quickly.

@ajantha-bhat
Copy link
Copy Markdown
Member Author

How long have we had this deprecated @ajantha-bhat? Are you aware of any compatibility guarantees or any timeline we might be bound to before fully removnig this?

I am aware that we deprecate things and wait for another minor release (like things deprecated in 12.x can be removed in 13.x)

This one was deprecated in FEB 2020. So, I think we can safely remove it as we have alternate implementation.

@ajantha-bhat
Copy link
Copy Markdown
Member Author

Handled by #5639

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants