4paradigm / 4paradigm/OpenMLDB

Add const constraints for udf input pointers

Offen
#2,351 1 Kommentar 0 Reaktionen 1 zugewiesene Person Beansprucht von @hezean Auf GitHub ansehen
enhancement execute-engine udf
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.