4paradigm / 4paradigm/OpenMLDB
derived SqlNode/PlanNode/... may incomplete due to missing override for `Equals` method
- 主要言語
- C++
- スター
- 1.7k
- フォーク
- 331
- 平均マージ
- 12日 12時間
- マージ済み PR（30日）
- 1
説明
e.g
- DeployPlanNode: https://github.com/4paradigm/openmldb/blob/836b34119249ff9b667130b125c88395ef18ee4f/hybridse/include/node/plan_node.h#L477
- DeployNode: https://github.com/4paradigm/openmldb/blob/836b34119249ff9b667130b125c88395ef18ee4f/hybridse/include/node/sql_node.h#L2207
_Originally posted by @aceforeverd in https://github.com/4paradigm/OpenMLDB/pull/1283#discussion_r811867541_
コントリビューションガイド
調査の方向性
The issue points to two derived classes (DeployPlanNode and DeployNode) in hybridse/include/node/plan_node.h and sql_node.h that may be missing overrides for the `Equals` method. First, examine the base class hierarchy to understand the expected `Equals` signature and semantics. Then, check if other derived classes provide overrides as examples. Implement the missing overrides, ensuring they compare all relevant member variables. Run existing tests related to node equality to verify correctness.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100