Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[QP-6630] Where절에서 사용한 컬럼에 대한 정보를 제공합니다. #82

Merged
merged 5 commits into from
May 21, 2024

Conversation

challenger71498
Copy link
Contributor

개요

  • Where 절에서 사용한 컬럼에 대한 정보를 얻을 수 있도록 추가합니다.
  • 실제 조회 시 사용하는 테이블에 포함되지 않는 컬럼이므로, 이름을 IndirectColumn으로 정했습니다.

현재 구현은 변경을 최소화하는 것을 최우선으로 하였습니다.

향후 QSI의 Analysis쪽 구현을 변경하여, 레이어를 하나 더 추가하여 관심사를 더욱 분리하고자 합니다.
자세한 구현 내용은 추후 컨플루언스 작성 후 해당 PR에 댓글로 달아놓도록 하겠습니다.

변경점

주요 사항

  • QsiTableStructureIndirectColumn을 추가합니다.
  • QsiTableAnalyzer에서 IndirectColumn을 조회합니다.
    • BuildDerivedTableStructure 메서드에서, where 절에서 사용하는 컬럼들을 IndirectColumns에 저장합니다.
    • TableCompileContextIsIndirectSource라는 플래그를 하나 추가합니다.
    • 해당 플래그가 참일 경우,
      • ResolveColumnsInExpression 메서드에서 TableExpression을 대상으로 컬럼을 가져올 때,
      • 추가적으로 해당 테이블의 IndirectColumn들까지 가져오도록 합니다.

기타

  • Debugger에서 IndirectColumn를 볼 수 있도록 합니다.

이슈

QP-6630

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

Copy link
Contributor

@tony-jang tony-jang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@challenger71498 challenger71498 merged commit d6655e8 into main May 21, 2024
8 of 20 checks passed
@challenger71498 challenger71498 deleted the feature/QP-6630-2 branch May 21, 2024 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants