yugabyte/yugabyte-db

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

Open

#22 246 ouverte le 3 mai 2024

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C (1 003 forks)batch import
area/ysqlgood first issuekind/enhancementpriority/mediumstatus/awaiting-triage

Métriques du dépôt

Stars
 (8 229 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (92 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez le commentaire à la ligne 243 dans src/postgres/src/include/access/amapi.h pour comprendre les ajouts de propriétés nécessaires, puis mettez à jour les fonctions liées à IndexAMProperty dans le même fichier pour inclure les nouvelles propriétés introduites dans IndexAmRoutine.
Stack technique
cpostgresql
Domaine
backenddatabase
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CPostgreSQL index access method internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40