4paradigm / 4paradigm/OpenMLDB

Add const constraints for udf input pointers

Aperta
#2,351 1 commento 0 reazioni 1 assegnatario Rivendicata da @hezean Vedi su GitHub
enhancement execute-engine udf
Lingua principale
C++
Stelle
1.7k
Fork
331
Merge medio
12g 12h
PR unite (30g)
1

Descrizione

For other udfs' signature, it would be better you can create a new PR and do the fix, if you are interested. Thanks.

_Originally posted by @zhanghaohit in https://github.com/4paradigm/OpenMLDB/pull/2262#discussion_r944313023_

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Esamina le firme UDF nella codebase, in particolare quelle modificate in PR #2262. Identifica i parametri puntatore di input che dovrebbero essere contrassegnati come const per motivi di sicurezza. Esamina le implementazioni UDF esistenti per comprendere il modello, quindi applica i vincoli const dove appropriato. I test devono garantire che le modifiche non interrompano le funzionalità UDF esistenti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
backend
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
30/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.