4paradigm / 4paradigm/OpenMLDB

refactor JitManager and HybridSeJitWrapper for simplify

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

説明

## Simplify
**Create** > **Init** , hides things like **InitSymbols**

## Error control

- returns `base::Status` on error, not `bool`

## Lifetime control: no manual delete in java

The `HybridSeJitWrapper` in hybridse-sdk is a proxy class to C pointer `HybridSeJitWrapper`. This requires extra config in swig so the proxy class will automatically release C resource from GC

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

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

調査の方向性

Look at the JitManager and HybridSeJitWrapper classes in the codebase, likely in a jit/ or sdk/ directory. Understand how InitSymbols and manual delete are currently handled. The goal is to refactor creation to hide InitSymbols, change error returns to base::Status, and configure SWIG for automatic lifetime management of the C pointer in the Java proxy. Check existing SWIG interface files for HybridSeJitWrapper.

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

評価

領域
backend, tooling
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

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

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