4paradigm / 4paradigm/OpenMLDB
non-standard print style in `PlanNode::Print`
- 主要言語
- C++
- スター
- 1.7k
- フォーク
- 331
- 平均マージ
- 12日 12時間
- マージ済み PR(30日)
- 1
説明
Issue tracker is **ONLY** used for reporting bugs. New features should be discussed on our discussion
- `"\t"` instead of `INDENT` marco is used in some node's `Print` function
- in some node's implementation, indent is wrongly provided. e.g https://github.com/4paradigm/OpenMLDB/blob/main/hybridse/src/node/plan_node.cc#L386
## Expected Behavior
## Current Behavior
## Possible Solution
## Steps to Reproduce
1.
2.
3.
4.
## Context (Environment)
## Detailed Description
## Possible Implementation
コントリビューションガイド
調査の方向性
The issue points to a specific file, hybridse/src/node/plan_node.cc, line 386, where an indent is incorrectly provided. Look for other uses of '\t' instead of the INDENT macro in PlanNode::Print functions across the codebase. Start by examining the Print functions in plan_node.cc and related files to identify non-standard print styles. Ensure changes maintain consistent formatting.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- backend
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100