[CBRD-26572] add SQL function 'UUID' by childyouth · Pull Request #6873 · CUBRID/cubrid

childyouth · 2026-02-26T11:30:56Z

http://jira.cubrid.org/browse/CBRD-26572
http://jira.cubrid.org/browse/CBRD-26573

Purpose

UUID 함수는 SYS_GUID와 달리 bit 형태의 반환값을 제공
임의값 기반의 UUIDv4, timestamp 기반의 UUIDv7 을 지원
UUIDv7은 timestamp 기반으로 생성되기에 index로 사용할 경우 한 개 페이지에 집중되어 삽입, 페이지 분기를 최소화할 수 있음
bit 형태로 반환 시 string 형태 대비 저장 공간을 2배 효율적으로 관리 가능

Implementation

UUID의 인자값으로 (1,2,3,5,6,8) 이 들어오는 경우 오류 생성
- 인자값 0, 4에 대해 UUIDv4 값 생성
- 인자값 7에 대해 UUIDv7 값 생성
함수 키워드로 UUID 추가
XASL_STATE 초기화 시 millisecond 추가
THREAD_ENTRY에 시간정보 추가
- XASL_STATE의 시간정보로 갱신
THREAD_ENTRY와 XASL_STATE의 시간정보를 바탕으로 동일 statement에서의 monotonic을 고려한 UUIDv7 생성

Remarks

: arg (int) : returns bit(128) : support UUIDv4 / UUIDv7 : uuid(7) = UUIDv7 : uuid(), uuid(4), uuid( int ) = UUIDv4

childyouth · 2026-02-26T11:35:09Z

+    case PT_SYS_GUID:
+      /* 
+       * constant folding for this expression is never performed : is always resolved on server
+       * PT_SYS_GUID and PT_UUID can be used as an attribute's default function
+       * We must perform a semantic check to ensure that the attribute domain and the default function type are compatible.
+       * During semantic checking, flag.do_not_fold is not considered; therefore, a dummy value is returned.
+       * Any PT_EXPR that contains PT_SYS_GUID or PT_UUID must always have flag.do_not_fold set to true.
+       */
+      db_make_string (result, "0123456789ABCDEF0123456789ABCDEF");
+      return 1;
+    case PT_UUID:
+      db_make_bit (result, GUID_STANDARD_BYTES_LENGTH * 8, "0123456789ABCDEF", GUID_STANDARD_BYTES_LENGTH * 8);
+      return 1;


해당 코드는 DEFAULT 지원 시 필요합니다.

더미 데이터로 UUID 및 SYS_GUID를 수행하여 column type과 function return domain의 호환을 확인합니다.
더미 데이터의 값이 조회될 경우는 없습니다.

Copilot

Pull request overview

Adds a new SQL built-in function UUID to generate UUID values as BIT(128), with support for UUIDv4 (random) and UUIDv7 (time-ordered) and statement-level monotonic behavior for v7.

Changes:

Introduces UUID parsing/type-checking and execution paths, including new arithmetic opcodes for UUIDv4/UUIDv7.
Implements UUID generation helpers (UUIDv4 hex for SYS_GUID() compatibility; UUIDv4/v7 binary for UUID()).
Extends query execution state to carry millisecond epoch time and adds per-thread UUIDv7 monotonic state.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/thread/thread_entry.hpp	Adds per-thread UUIDv7 monotonic state + inline accessors.
src/thread/thread_entry.cpp	Initializes UUIDv7 per-thread state in thread entry constructor.
src/storage/storage_common.h	Adds new opcode identifiers `T_UUID4` / `T_UUID7`.
src/query/string_opfunc.h	Adds UUID version enum/constants and new UUID function declarations.
src/query/string_opfunc.c	Implements UUIDv4 (hex) and UUID(v4/v7) binary generation helpers.
src/query/query_executor.h	Adds `sys_epochtime_ms` to `val_descr` for UUIDv7 epoch ms precision.
src/query/query_executor.c	Initializes/copies `sys_epochtime_ms` in XASL state.
src/query/fetch.c	Executes new UUID opcodes and routes `SYS_GUID` to `db_uuidv4`.
src/parser/xasl_generation.c	Generates regu vars for `UUID()` / `UUID(int)` and adds semantic error for unsupported literal versions.
src/parser/type_checking.c	Adds overload/type rules for `UUID` and domain precision for BIT(128).
src/parser/parser_message.h	Adds message id for unsupported UUID versions.
src/parser/parse_tree_cl.c	Adds printing support for `uuid(...)` and opcode name mapping.
src/parser/parse_tree.h	Adds `PT_UUID` node/operator plumbing.
src/parser/keyword.c	Registers `uuid` as a function keyword.
src/parser/csql_grammar.y	Parses `UUID` function with 0 or 1 argument and disables caching/folding.
src/optimizer/query_graph.c	Marks `UUID` as a heavy expression for ranking.
msg/ko_KR.utf8/cubrid.msg	Adds Korean semantic error message for unsupported UUID versions.
msg/en_US.utf8/cubrid.msg	Adds English semantic error message for unsupported UUID versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

childyouth · 2026-02-27T05:05:28Z

UUID 함수의 인자값으로 컬럼을 받을 수 없는 문제가 있어 해결 후 다시 open하도록 하겠습니다.
죄송합니다.

childyouth · 2026-02-27T07:40:26Z

+  T_UUID,
+  T_UUID4,
+  T_UUID7,


T_UUID4, T_UUID7은 컬럼을 인자로 받기 위해 T_UUID로 통합되었기에 당장 사용되지는 않습니다.

하지만 DEFAULT 지원 시 필요할 수 있어 우선 남겨두고자 합니다.

현재 default 함수는 to_char를 제외하면 인자값을 허용하지 않습니다.

이를 우회하기 위해 default 지정 시 T_UUID4, T_UUID7을 사용하는 방식을 고려하고 있습니다.
(default에선 컬럼이 인자로 올 수 없습니다.)

언제쯤 필요하게 될까요?
추후 작업을 위해 미리 넣어 두는 것이라면 #ifdef 등으로 묶어 두는 것은 어떤가요?

default 처리 PR은 곧 진행해야 할 것 같으니, 그냥 두어도 괜찮을 듯 합니다.

@ctshim @beyondykk9

PR 순서로 UUID_FORMAT -> SYS_GUID DEFAULT & UUID DEFAULT 를 생각하고 있습니다.

네 알겠습니다.

childyouth · 2026-03-10T06:41:23Z

/run all

@kangmin5505

@kangmin5505

childyouth · 2026-03-10T08:40:45Z

/run all

childyouth · 2026-03-10T10:41:26Z

/run all

childyouth · 2026-03-11T03:51:09Z

test_sql 실패는 develop merge 시 해결되는 케이스입니다.

kangmin5505 · 2026-03-12T05:43:47Z

SymbolStack.java 파일에 함수명 키워드 등록이 필요해 보입니다.

greptile-apps · 2026-03-13T08:05:22Z

_{Last reviewed commit: 2f7281e}

greptile-apps · 2026-03-13T09:57:39Z

_{Last reviewed commit: 36877ca}

childyouth · 2026-03-13T10:20:57Z

SymbolStack.java 파일에 함수명 키워드 등록이 필요해 보입니다.

@kangmin5505 @hyunikn

SymbolStack.java 파일에 함수명을 추가하였으나 실행은 할 수 없습니다.

PL/CSQL에서 bit 타입은 지원하지 않으며 return 타입이 bit 형 이기 때문에 다음과 같이 실행불가한 상태입니다.
(https://www.cubrid.org/manual/ko/11.4/pl/plcsql_overview.html#types)

csql> create procedure uuid_print()
as 
id varchar; 
begin
select uuid(7) into id;
dbms_output.put_line('uuid : ' || id);
end;

In line 3, column 1,

ERROR: Stored procedure compile error: the SELECT statement contains a column $anonymous1 of an unsupported type BIT


0 command(s) successfully processed.

greptile-apps · 2026-03-13T10:23:46Z

_{Last reviewed commit: e49130e}

greptile-apps · 2026-03-17T09:16:14Z

_{Last reviewed commit: 46a902b}

greptile-apps · 2026-03-17T09:16:18Z

+  T_UUID,
+  T_UUID4,
+  T_UUID7,
+  T_UUID_FORMAT,


T_UUID4, T_UUID7 — XASL 생성 경로가 없는 데드 코드(Dead Code)

T_UUID4와 T_UUID7가 OPERATOR_TYPE enum에 추가되고 fetch.c에도 case 핸들러가 구현되어 있습니다. 그러나 xasl_generation.c에서 PT_UUID를 변환할 때는 오직 T_UUID만 생성됩니다:

case PT_UUID: regu = pt_make_regu_arith (r1, r2, NULL, T_UUID, domain); // T_UUID4/T_UUID7는 생성 안됨 break;

즉, 런타임에 T_UUID4나 T_UUID7 opcode가 실행될 경로가 존재하지 않습니다. 이로 인해 발생하는 문제:

XASL 직렬화 호환성 리스크: OPERATOR_TYPE enum에 새 값이 삽입되면, 이후 모든 enum 값이 shift됩니다. XASL 플랜 캐시 등 enum 값을 직렬화하는 경로에서 T_SLEEP 이후 opcode들의 값이 변경되어 기존 캐시와의 호환성 문제가 생길 수 있습니다.

유지보수 혼란: 실제로 실행되지 않는 코드 경로가 존재하면 향후 리팩토링 시 혼란을 야기합니다.

이 PR에서 실제로 사용하지 않을 예정이라면 T_UUID4, T_UUID7을 enum과 fetch.c에서 제거하는 것을 권장합니다.

default 에서 사용할 수 도 있어 추후 PR에서 판단할 예정입니다.

greptile-apps · 2026-03-17T09:16:19Z

+    }
+
+  /* Save updated state back to thread */
+  thread_set_uuidv7_state (thread_p, last_ms, seq);
+
+  /* Generate random bytes for the lower part (bytes 7-15) */
+  error_code = crypt_generate_random_bytes ((char *) (out_bytes + 7), GUID_STANDARD_BYTES_LENGTH - 7);
+  if (error_code != NO_ERROR)
+    {
+      return error_code;
+    }


UUIDv7 스레드 상태 업데이트 순서 — 오류 발생 시 시퀀스 소모 문제

thread_set_uuidv7_state 호출이 crypt_generate_random_bytes 호출보다 먼저 이루어지고 있습니다:

/* Save updated state back to thread */ thread_set_uuidv7_state (thread_p, last_ms, seq); // ← 상태 먼저 커밋 /* Generate random bytes for the lower part (bytes 7-15) */ error_code = crypt_generate_random_bytes (...); if (error_code != NO_ERROR) { return error_code; // ← 이미 상태는 진행된 상태 }

crypt_generate_random_bytes가 실패하면, UUID는 실제로 생성되지 않았음에도 불구하고 thread_p->uuidv7_last_ms와 thread_p->uuidv7_seq는 이미 증가한 상태로 남습니다. 이후 정상적인 재시도 시 시퀀스 번호가 불필요하게 소모됩니다.

crypt_generate_random_bytes가 일상적으로 실패하는 케이스는 드물지만, 고부하 환경이나 RNG 초기화 실패 상황에서는 연속적인 오류 시 시퀀스가 빠르게 소모될 수 있습니다. 상태 업데이트를 성공 확인 이후로 이동하는 것이 더 견고한 설계입니다:

/* Generate random bytes first, THEN commit state */ error_code = crypt_generate_random_bytes ((char *) (out_bytes + 7), GUID_STANDARD_BYTES_LENGTH - 7); if (error_code != NO_ERROR) { return error_code; } /* Save updated state back to thread only on success */ thread_set_uuidv7_state (thread_p, last_ms, seq);

실패한다면 해당 statement가 취소될 것이기에 불필요한 염려라고 생각합니다.

greptile-apps · 2026-03-17T09:16:20Z

 		       || node->info.expr.op == PT_DECRYPT || node->info.expr.op == PT_BIN
 		       || node->info.expr.op == PT_MD5 || node->info.expr.op == PT_SHA_ONE
 		       || node->info.expr.op == PT_SPACE || node->info.expr.op == PT_PRIOR
+		       || node->info.expr.op == PT_UUID_FORMAT || node->info.expr.op == PT_UUID


PT_UUID_FORMAT — 데드 코드 조건 포함

PT_UUID_FORMAT이 unary-style op 목록 조건에 포함되어 있습니다. 하지만:

csql_grammar.y에 uuid_format() SQL 문법 규칙이 없어서 파서 단에서 생성되지 않습니다.

이 블록 이후의 case PT_UUID_FORMAT: XASL 생성 코드가 없어, 만약 도달한다면 오류가 됩니다.

fetch.c에서는 #if 0으로 비활성화되어 있습니다.

fetch.c에서 #if 0으로 격리된 것과 동일하게, 이 조건에서도 PT_UUID_FORMAT을 제거하거나 #if 0으로 감싸는 것이 코드 일관성 측면에서 바람직합니다:

Suggested change

|| node->info.expr.op == PT_UUID_FORMAT || node->info.expr.op == PT_UUID

|| node->info.expr.op == PT_UUID

default에서 판단할 예정입니다.

childyouth · 2026-03-17T11:32:54Z

greptile 코드 리뷰 영향인 d2a514d ~ 46a902b 까지 변경점이 크지 않고 feature 브랜치에 병합되므로 codereview 재요청은 수행하지 않겠습니다.

github-actions · 2026-03-19T06:17:37Z

✅ TC Branch Finalized for cubrid-testcases

Engine PR was merged.

Cleanup Results:

TC develop branch is ready for the next PR.

github-actions · 2026-03-19T06:17:37Z

✅ TC Branch Finalized for cubrid-testcases-private-ex

Engine PR was merged.

Cleanup Results:

TC develop branch is ready for the next PR.

childyouth added 5 commits February 6, 2026 19:44

function uuid()

661d5a0

: arg (int) : returns bit(128) : support UUIDv4 / UUIDv7 : uuid(7) = UUIDv7 : uuid(), uuid(4), uuid( int ) = UUIDv4

transfrom into regu variable

8e82156

apply codestyle. remove unused functions

b7dbee9

remove unused functions

96d5b7f

error for UUID(1) ~ UUID(8) except UUID(4) and UUID(7)

f7a1ee6

childyouth requested review from airnet73, ctshim, hyunikn, jongmin-won and kangmin5505 February 26, 2026 11:30

childyouth self-assigned this Feb 26, 2026

childyouth requested review from beyondykk9, hornetmj and shparkcubrid as code owners February 26, 2026 11:30

childyouth commented Feb 26, 2026

View reviewed changes

kangmin5505 requested review from Copilot and kwangsoochae February 26, 2026 23:51

Copilot started reviewing on behalf of kangmin5505 February 26, 2026 23:52 View session

Copilot AI reviewed Feb 27, 2026

View reviewed changes

Comment thread src/parser/xasl_generation.c Outdated

Comment thread src/query/string_opfunc.c Outdated

Comment thread msg/en_US.utf8/cubrid.msg Outdated

kwangsoochae reviewed Feb 27, 2026

View reviewed changes

Comment thread src/parser/xasl_generation.c Outdated

kwangsoochae approved these changes Feb 27, 2026

View reviewed changes

Apply suggestions from code review

ff0cfc6

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

childyouth marked this pull request as draft February 27, 2026 05:05

childyouth added 3 commits February 27, 2026 16:29

handle column for uuid's input

2f16c29

Merge remote-tracking branch 'origin/CBRD-26572' into CBRD-26572

4f2c44b

define PT_OP_TYPE.PT_UUID_FORMAT

80404a4

childyouth commented Feb 27, 2026

View reviewed changes

apply codestyle

8497823

childyouth marked this pull request as ready for review February 27, 2026 07:46

Merge remote-tracking branch 'upstream/feature/uuid' into CBRD-26572

d2a514d

apply codereview (remove sys_epochtime_ms)

9794548

@kangmin5505

apply codereview (assert for check seq max)

622fc1b

Merge remote-tracking branch 'upstream/develop' into CBRD-26572

4d11f91

hyunikn reviewed Mar 12, 2026

View reviewed changes

Comment thread src/parser/type_checking.c Outdated

hyunikn reviewed Mar 12, 2026

View reviewed changes

Comment thread src/query/fetch.c Outdated

hyunikn reviewed Mar 12, 2026

View reviewed changes

Comment thread src/query/fetch.c

apply codereview

2f7281e

greptile-apps Bot reviewed Mar 13, 2026

View reviewed changes

Comment thread src/query/fetch.c

Comment thread src/query/fetch.c Outdated

Comment thread src/query/fetch.c

Comment thread src/query/string_opfunc.h

Comment thread src/query/string_opfunc.c

apply codereview

36877ca

apply codereview

e49130e

CUBRID deleted a comment from greptile-apps Bot Mar 13, 2026

greptile-apps Bot reviewed Mar 13, 2026

View reviewed changes

Comment thread src/query/string_opfunc.c Outdated

Comment thread src/query/fetch.c

kangmin5505 approved these changes Mar 14, 2026

View reviewed changes

apply codereview (assert for type checking of uuidv7_seq)

46a902b

greptile-apps Bot reviewed Mar 17, 2026

View reviewed changes

childyouth merged commit c6228b3 into CUBRID:feature/uuid Mar 19, 2026
8 checks passed

	\|\| node->info.expr.op == PT_UUID_FORMAT \|\| node->info.expr.op == PT_UUID
	\|\| node->info.expr.op == PT_UUID

Conversation

childyouth commented Feb 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose

Implementation

Remarks

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

childyouth commented Feb 27, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

childyouth commented Mar 10, 2026

Uh oh!

childyouth commented Mar 10, 2026

Uh oh!

childyouth commented Mar 10, 2026

Uh oh!

childyouth commented Mar 11, 2026

Uh oh!

Uh oh!

kangmin5505 commented Mar 12, 2026

Uh oh!

Uh oh!

Uh oh!

greptile-apps Bot commented Mar 13, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

greptile-apps Bot commented Mar 13, 2026

Uh oh!

childyouth commented Mar 13, 2026

Uh oh!

greptile-apps Bot commented Mar 13, 2026

Uh oh!

Uh oh!

Uh oh!

greptile-apps Bot commented Mar 17, 2026

Uh oh!

greptile-apps Bot Mar 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot Mar 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot Mar 17, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

childyouth commented Mar 17, 2026

Uh oh!

Uh oh!

github-actions Bot commented Mar 19, 2026

Uh oh!

childyouth commented Feb 26, 2026 •

edited

Loading