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.

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

Update bookmark manager API #410

Merged
merged 4 commits into from
Nov 25, 2022
Merged

Update bookmark manager API #410

merged 4 commits into from
Nov 25, 2022

Conversation

fbiville
Copy link
Contributor

No description provided.

@fbiville fbiville marked this pull request as ready for review November 23, 2022 14:25
@fbiville fbiville changed the title WIP update bookmark manager API Update bookmark manager API Nov 23, 2022
As explained in ADR 011, Fabric combines bookmarks when multiple
databases are involved in a single transaction so there is no need
on the driver side to track databases.
Copy link
Contributor

@bigmontz bigmontz left a comment

Choose a reason for hiding this comment

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

🔖 👔

neo4j/bookmarks.go Outdated Show resolved Hide resolved
neo4j/bookmarks.go Outdated Show resolved Hide resolved
neo4j/bookmarks.go Outdated Show resolved Hide resolved
fbiville and others added 3 commits November 24, 2022 11:40
Co-authored-by: Antonio Barcélos <antonio.barcelos@neo4j.com>
Co-authored-by: Antonio Barcélos <antonio.barcelos@neo4j.com>
Co-authored-by: Antonio Barcélos <antonio.barcelos@neo4j.com>
@fbiville fbiville merged commit 576d8e0 into 5.0 Nov 25, 2022
@fbiville fbiville deleted the bm_update branch November 25, 2022 14:06
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.

2 participants