4paradigm / 4paradigm/OpenMLDB

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

未关闭
#660 1 条评论 0 个 reaction 已指派 1 人 已被 @aceforeverd 认领 在 GitHub 查看
enhancement execute-engine
主要语言
C++
星标
1.7k
派生
331
平均合并
12 天 12 小时
30 天内合并 PR
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 摘要。