4paradigm / 4paradigm/OpenMLDB

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

Ouverte
#660 1 commentaire 0 réactions 1 personne assignée Réclamée par @aceforeverd Voir sur GitHub
enhancement execute-engine
Langage dominant
C++
Étoiles
1.7k
Forks
331
Merge moyen
12 j 12 h
PR mergées (30 j)
1

Description

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.