4paradigm / 4paradigm/OpenMLDB

non-standard print style in `PlanNode::Print`

オープン
#660 コメント 1 件 リアクション 0 件 担当者 1 名 @aceforeverd が担当を希望しています GitHub で見る
enhancement execute-engine
主要言語
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

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

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