yugabyte/yugabyte-db

ysql: fix uses of parser_ybc_beta_feature

Open

#10.263 aperta il 12 ott 2021

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C (1003 fork)batch import
area/ysqlgood first issuehelp wantedkind/enhancementpriority/medium

Metriche repository

Star
 (8229 star)
Metriche merge PR
 (Merge medio 17g 21h) (92 PR mergiate in 30 g)

Descrizione

Jira Link: DB-1261 There are multiple issues with the individual beta feature flags. I believe the core issue is lack of understanding of parser_ybc_beta_feature. See the definition:

#define parser_ybc_beta_feature(pos, feature, has_own_flag) \
    check_beta_feature(pos, yyscanner, has_own_flag ? "FLAGS_ysql_beta_feature_" feature : NULL, feature)

If has_own_flag is true, there should be a corresponding flag declared, and feature should not have whitespace. It looks like this macro was used properly until commit a3c112233b621164d4367420c9db619aae7cf1c5. Bad uses include commit bc28a34342f94a0ee89391a98ccd1c174d5d0ba7 (my fault, cc: @robertsami) and 830ba9964031fbcbff9e1fb07cf8a215cfd7c7fa (PR #9650, cc: @fizaaluthra).

Guida contributor

Direzione di ricerca
Identificare tutti gli usi della macro parser ybc beta feature nel codebase, verificare che has own flag sia impostato correttamente e che la stringa feature non contenga spazi, quindi correggere gli usi impropri.
Tech stack
c
Dominio
backenddatabase
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70