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

[CBRD-25610] null value was removed on cci #684

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

jongmin-won
Copy link
Contributor

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

Purpose

CTP(sql_by_cci) 결과가 잘못 나오는 문제가 발생하여 이를 수정했습니다.

PR(#682) 에서 상수 sql의 값을 변경할 수 없다고 생각하여 새로운 상수 sql2를 추가했으나, cci에서 PLCSQL을 수행하는 중 기존 null 값이 제거되는 문제가 발생했습니다.

따라서, 새로운 상수 sql2를 제거하고 기존 sql로 되돌렸습니다.

Implementation

N/A

Remarks

N/A

@jongmin-won jongmin-won self-assigned this Oct 8, 2024
@hyunikn
Copy link
Contributor

hyunikn commented Oct 8, 2024

이 변경 전후에 CTP의 sql_on_cci 테스트에서 fail 갯수는 어떻게 되나요?
로컬에서 돌리면 한시간 반 정도 걸렸던 것 같은데 확인해 볼 필요가 있을 것 같습니다.

@jongmin-won
Copy link
Contributor Author

jongmin-won commented Oct 8, 2024

CTP 변경 전후 sql_on_cci 테스트 fail 결과 공유 드립니다.

  1. CTP 변경 전
  • CTP 버전: develop
  • CUBRID 엔진 버전: develop
  • Fail : 2,068
  • Success : 15,145
  • Total :17,213
  1. CTP 변경 후
  1. 기타
  • 코어 덤프 발생: CTP 변경 여부와 관계없이 cci 테스트 중 발생
    -- 테스트 케이스: sql/_08_javasp/cases/case_cte_01.sql
    -- bug report : CBRD-24658

@jongmin-won jongmin-won merged commit c893616 into CUBRID:develop Oct 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants