4paradigm / 4paradigm/OpenMLDB
Add const constraints for udf input pointers
- Vorherrschende Sprache
- C++
- Sterne
- 1.7k
- Forks
- 331
- Ø Merge
- 12 T. 12 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
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_
Beitragsleitfaden
Rechercherichtung
Sieh dir die UDF-Signaturen in der Codebasis an, insbesondere die in PR #2262 geänderten. Identifiziere Eingabezeigerparameter, die aus Sicherheitsgründen als const markiert werden sollten. Überprüfe die vorhandenen UDF-Implementierungen, um das Muster zu verstehen, und wende const-Einschränkungen an, wo dies angemessen ist. Beim Testen muss sichergestellt werden, dass die Änderungen die vorhandene UDF-Funktionalität nicht beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 30/100