Skip to content

[CBRD-27439] Exposure of hidden columns in PL/CSQL containing CTE queries - #7944

Open
beyondykk9 wants to merge 4 commits into
CUBRID:developfrom
beyondykk9:CBRD-27439
Open

beyondykk9 wants to merge 4 commits into
CUBRID:developfrom
beyondykk9:CBRD-27439

Conversation

@beyondykk9

Copy link
Copy Markdown
Contributor

http://jira.cubrid.org/browse/CBRD-27439

Purpose

[CBRD-27334]에서 order by 처리를 위해 생성된 히든 컬럼이 PL/CSQL에서 select list에 노출되어 시멘틱 에러 발생하는 오류 수정

Implementation

PL에서 order by로 생성된 히든 컬럼이 포함된 static sql에 대해 히든 컬럼을 select list에 노출하지 않도록 parser코드 수정

Remarks

11.4 이하 버전에서는 WITH CTE쿼리를 머지하지 않으므로 발생하지 않음.

@beyondykk9 beyondykk9 added this to the guava milestone Sep 15, 2026
@beyondykk9 beyondykk9 self-assigned this Sep 15, 2026
@beyondykk9 beyondykk9 added bug regression legacy inherited legacy issues labels Sep 15, 2026
@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown

❌ TC Merge Gate — Merge Blocked

One or more TC PRs are still open. Please merge or close them before merging this PR.

TC Repositories & Branches:

  • cubrid-testcases: TC PR tc/pr-7944 is open (draft) — must be merged or closed first
  • cubrid-testcases-private-ex: TC PR tc/pr-7944 is open (draft) — must be merged or closed first

Steps to unblock:

  1. Merge or close all TC PRs listed above.
  2. Re-run this check: Actions tab → TC Merge Gate → Re-run failed jobs

@github-actions

Copy link
Copy Markdown

🧪 TC Test Environment Ready

CircleCI Testing:

  • CircleCI will automatically test using the branches below.

TC Repositories & Branches:

Next Steps:

  1. Wait for CircleCI tests to complete
  2. If CircleCI tests failed, please check the test results and fix the issues.
  3. When ready to merge this PR, please merge the TC PR first, then merge this PR.

@greptile-apps

greptile-apps Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Retrigger

이전 리뷰의 두 메모리 관리 문제가 현재 코드에서 해소되었으며, 이번 후속 변경은 병합해도 안전해 보입니다.

Reviews (2) · Last reviewed commit: "review feedback to code"

Comment thread src/parser/parse_tree_cl.c
Comment thread src/parser/parse_tree_cl.c Outdated
@beyondykk9

Copy link
Copy Markdown
Contributor Author

@greptile 리뷰해줘

@beyondykk9

Copy link
Copy Markdown
Contributor Author

/run all

@beyondykk9

Copy link
Copy Markdown
Contributor Author

/run all

1 similar comment
@beyondykk9

Copy link
Copy Markdown
Contributor Author

/run all

@beyondykk9

Copy link
Copy Markdown
Contributor Author

/run all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug legacy inherited legacy issues regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant