Skip to content

[CBRD-26519] modify TC - auto_increment spec changed#2505

Merged
childyouth merged 3 commits intoCUBRID:developfrom
childyouth:CBRD-26519
Apr 1, 2026
Merged

[CBRD-26519] modify TC - auto_increment spec changed#2505
childyouth merged 3 commits intoCUBRID:developfrom
childyouth:CBRD-26519

Conversation

@childyouth
Copy link
Copy Markdown
Contributor

refer to CUBRID/cubrid#6815

auto_increment를 테이블 당 한 개로 제한하는 스펙 변경이 있어 이를 허용하던 TC의 스크립트와 정답을 변경합니다.

@childyouth childyouth self-assigned this Mar 11, 2026
@childyouth
Copy link
Copy Markdown
Contributor Author

childyouth commented Mar 23, 2026

@ssihil @kwonhoil @kiho-um

sql/_17_sql_extension2/_02_full_test/_08_other_functions/_03_last_insert_id/cases/last_insert_id_005.sql 에 대해 의견을 여쭤보고자 합니다.


해당 TC는 두 개의 AUTO_INCREMENT를 설정한 테이블에 삽입을 수행합니다.

이번 스펙 변경으로 AUTO_INCREMENT는 한 개로 제한하기에 오류를 발생하도록 수정을 하였습니다.

이로 인해 insert가 일어나지 않아 이전 실행 sql인 last_insert_id_004.sqllast_insert_id()인 6이 결과값으로 보여집니다.
last_insert_id_005.answer는 단독 실행 기준으로 작성되었기에 NULL을 기대하여 TC fail이 일어나게 됩니다. (링크)


AUTO_INCREMENT가 테이블 당 한 개로 제한되면서 해당 TC의 의미가 사라진 것 같아 제거를 하는 것이 맞지 않을까 합니다.

해당 TC를 제거해도 괜찮을까요?

@childyouth childyouth requested review from kwonhoil and ssihil March 23, 2026 11:21
@ssihil
Copy link
Copy Markdown
Collaborator

ssihil commented Mar 24, 2026

@ssihil @kwonhoil @kiho-um

sql/_17_sql_extension2/_02_full_test/_08_other_functions/_03_last_insert_id/cases/last_insert_id_005.sql 에 대해 의견을 여쭤보고자 합니다. 해당 TC는 두 개의 AUTO_INCREMENT를 설정한 테이블에 삽입을 수행합니다.

이번 스펙 변경으로 AUTO_INCREMENT는 한 개로 제한하기에 오류를 발생하도록 수정을 하였습니다.

이로 인해 insert가 일어나지 않아 이전 실행 sql인 last_insert_id_004.sqllast_insert_id()인 6이 결과값으로 보여집니다. last_insert_id_006.answer는 단독 실행 기준으로 작성되었기에 NULL을 기대하여 TC fail이 일어나게 됩니다. (링크) AUTO_INCREMENT가 테이블 당 한 개로 제한되면서 해당 TC의 의미가 사라진 것 같아 제거를 하는 것이 맞지 않을까 합니다.

해당 TC를 제거해도 괜찮을까요?

last_insert_id() 값이 일정하게 유지가 안된다면 TC를 삭제하는 것이 맞을 것 같습니다.

@childyouth
Copy link
Copy Markdown
Contributor Author

@kwonhoil @ssihil @kiho-um

더 이상 필요없다고 판단된 테스트케이스 last_insert_id_005.sqllast_insert_id_006.sql 을 제거했습니다.

감사합니다.

@childyouth childyouth merged commit 57b6110 into CUBRID:develop Apr 1, 2026
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.

4 participants