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-25352] Add a user schema to your stored procedures for consistency with other objects. #677

Conversation

jongmin-won
Copy link
Contributor

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

Built-in PL/CSQL을 호출할 때는 'dbms_output.'을 붙여야 정상 실행됩니다.
ex) 'call get_line;' -> 'call dbms_output.get_line;'

…_output.' to ensure normal execution. For example, 'call get_line;' -> 'call dbms_output.get_line;'
@jongmin-won jongmin-won self-assigned this Jul 24, 2024
@jongmin-won jongmin-won marked this pull request as ready for review July 24, 2024 11:02
@jongmin-won jongmin-won requested review from hgryoo and hyunikn July 24, 2024 11:03
@swi0110 swi0110 requested review from ssihil and kwonhoil July 25, 2024 00:31
@hyunikn
Copy link
Contributor

hyunikn commented Jul 25, 2024

이렇게 변경 후 테스트가 정상 진행 되나요.

@swi0110
Copy link
Contributor

swi0110 commented Jul 25, 2024

@hyunikn
혹시 이 내용이 아래의 이슈하고도 연관이 있을까요?
http://jira.cubrid.org/browse/CBRD-24820

@hyunikn
Copy link
Contributor

hyunikn commented Jul 25, 2024

@hyunikn 혹시 이 내용이 아래의 이슈하고도 연관이 있을까요? http://jira.cubrid.org/browse/CBRD-24820

네. 관련이 있습니다.
종민님 SP에 user schema 적용 구현이 반영되면 (CUBRID/cubrid#5258)
put_line 등 6개 dbms_output 메소드에 dbms_output 을 반드시 붙여야 동작하게 됩니다.

@jongmin-won
Copy link
Contributor Author

jongmin-won commented Jul 25, 2024

@hyunikn
네 CTP 툴 수정이 반영되면 테스트가 정상 진행됩니다.

추가로 금일 QA팀과 논의한 결과, PR을 반영하는 branch를 변경하기로 했습니다. (feature/plcsql -> develop)

develop로 변경 시 파일 하나를 더 수정해야 하므로, 해당 PR을 Close하고 새로운 PR을 재등록하도록 하겠습니다.

@jongmin-won jongmin-won deleted the feature/plcsql-CBRD-25352-testtools branch July 25, 2024 14:09
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.

3 participants