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-8036] PgTableAnalyzer.BuildTableFunctionStructure에서, 리턴 테이블에 컬럼 이름 있는 경우 그 이름 가져오기 #79

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

irwin-chequer
Copy link
Contributor

  1. 현재 Qsi.PostgreSql는 nuget의 PgQuery을 사용하는데, PgQuery는 테이블 함수를 포함하는 쿼리를 파싱하도록 하면, 만약 해당 테이블 함수가 리턴값으로 컬럼을 갖는 테이블을 리턴하면, 그 함수에 해당하는 노드가 갖는 Parameter 목록에 그 함수의 매개변수와 그 함수가 리턴하는 테이블의 컬럼들을 모두 포함해서 리턴합니다. (테이블 함수가 리턴값으로 컬럼을 갖는 테이블을 리턴하지 않는다면 Parameter 목록에 리턴값에 관한 정보가 포함되지는 않습니다.)

  2. 기존 Qsi.PostgreSql 은 이처럼 테이블 함수의 리턴값이 컬럼을 갖는 테이블인 경우에 관한 처리가 없어, 테이블 함수의 리턴 테이블에서 컬럼이름으로 값을 가져오려 할 경우 '그러한 컬럼이 없다'는 에러가 발생합니다. 이 PR은 그에 관한 구현을 추가하여 이 에러를 해결합니다.

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

@irwin-chequer irwin-chequer requested review from ryan-chae, tony-jang and mosty-kim and removed request for ryan-chae April 25, 2024 06:02
Copy link

@mosty-kim mosty-kim left a comment

Choose a reason for hiding this comment

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

어윈이 자세히 설명 해주셔서 이해할 수 있었습니다. 👍

@irwin-chequer irwin-chequer merged commit 81b479c into main Apr 26, 2024
20 of 21 checks passed
@irwin-chequer irwin-chequer deleted the feature/QP-8036 branch April 26, 2024 08:50
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.

2 participants