Skip to content

PGD: document new function "bdr.galloc_chunk_info()" (WIP) #6588

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

ibarwick
Copy link
Contributor

@ibarwick ibarwick commented Mar 6, 2025

What Changed?

This function removes the need to query galloc sequences allocated on the local node by referencing hidden rows by their CTID, which is really not something we want end users to be doing.

Add reference section, and update the section where we were telling end users to query individual tuples by CTID.

DOCS-1258.

@ibarwick ibarwick requested a review from a team as a code owner March 6, 2025 06:07
@djw-m djw-m force-pushed the dev/ib/BDR6144-galloc-chunk-values-func branch 2 times, most recently from 663d5b1 to cae5c7f Compare March 6, 2025 09:12
Copy link
Contributor

@djw-m djw-m left a comment

Choose a reason for hiding this comment

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

This is being targeted at 5.8 and 6.0? It's set to update on 5.7 so it needs to be rebased to the 5.8 branch at least.

ibarwick and others added 2 commits May 2, 2025 05:02
This function removes the need to query galloc sequences allocated
on the local node by referencing hidden rows by their CTID, which is
really not something we want end users to be doing.

DOCS-1258.
@djw-m djw-m force-pushed the dev/ib/BDR6144-galloc-chunk-values-func branch from 62a809a to fdaf5d3 Compare May 2, 2025 09:02
@@ -220,6 +220,25 @@ bdr.timestamp_to_timeshard(ts timestamptz)

- `ts` — Timestamp to use for the `timeshard` sequence generation.

### `bdr.galloc_chunk_info`
Copy link
Contributor

Choose a reason for hiding this comment

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

I suspect the positioning has removed the notes section from timestamp_to_timeshard

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