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.

Sign up for GitHub

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

Jump to bottom

[YSQL] Update IndexAMProperty API to be in sync with YB additions to IndexAmRoutine #22246

Open
1 task done
tanujnay112 opened this issue May 3, 2024 · 0 comments
Open
1 task done

[YSQL] Update IndexAMProperty API to be in sync with YB additions to IndexAmRoutine #22246

tanujnay112 opened this issue May 3, 2024 · 0 comments
Labels
area/ysql Yugabyte SQL (YSQL) good first issue This is a good issue to start contributing! kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue status/awaiting-triage Issue awaiting triage

Comments

@tanujnay112
Copy link
Contributor

tanujnay112 commented May 3, 2024 •
edited by jira bot

Jira Link: DB-11164

Description

We have added a few interface methods to IndexAMRoutine in access/amapi.h. However, this comment says that property additions to this struct must alos be reflected in the "property API". This would allow the PG functions pg_indexam_has_property, pg_index_has_property and pg_index_column_has_property to access these new properties.

This issue tracks this sync-up that needs to be done between the two structures as the linked comment suggests.

Issue Type

kind/enhancement

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.
@tanujnay112 tanujnay112 added good first issue This is a good issue to start contributing! area/ysql Yugabyte SQL (YSQL) status/awaiting-triage Issue awaiting triage labels May 3, 2024
@yugabyte-ci yugabyte-ci added kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue labels May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/ysql Yugabyte SQL (YSQL) good first issue This is a good issue to start contributing! kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue status/awaiting-triage Issue awaiting triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tanujnay112 @yugabyte-ci