4paradigm / 4paradigm/OpenMLDB

Add const constraints for udf input pointers

オープン
#2,351 コメント 1 件 リアクション 0 件 担当者 1 名 @hezean が担当を希望しています GitHub で見る
enhancement execute-engine udf
主要言語
C++
スター
1.7k
フォーク
331
平均マージ
12日 12時間
マージ済み PR(30日)
1

説明

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_

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

コードベース内の UDF シグネチャ、特に PR #2262 で変更されたものを確認します。安全性のために const としてマークすべき入力ポインターパラメーターを特定します。既存の UDF 実装を確認してパターンを理解し、適切な箇所に const 制約を適用します。テストでは、変更によって既存の UDF 機能が壊れないことを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp
領域
backend
issue の種類
リファクタリング
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
30/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。