[CBRD-26447] (Phase-1) Create/Alter invisible column #6687

childyouth · 2025-12-03T08:28:05Z

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

Purpose

INVISIBLE COLUMN

Invisible Column은 명시적으로 컬럼 이름이 나열되지 않은 명령에 대해 결과를 숨기도록 하는 컬럼 옵션을 말함.

Invisible Column을 설정함으로서 얕은 부분의 보안성을 향상 시킬 수 있음.

다른 버전의 어플리케이션에 대해 정지없이 컬럼 추가가 가능해 지원성을 높일 수 있음.

시스템 측면에서 보여줄 필요가 없는 컬럼에 대해 설정함으로서 명확성을 높일 수 있음.

Phase 1

CREATE TABLE, ALTER TABLE ADD COLUMN / MODIFY / CHANGE 를 통해 컬럼에 Invisible 옵션을 부여할 수 있음.

컬럼에 Invisible 옵션을 확인할 수 있도록 DESC, SHOW COLUMNS 및 시스템 카탈로그에 해당되는 정보를 표시할 수 있음.

Implementation

CREATE TABLE <table name> ( <col_definition>... )
<col_definition> ::= <col_name> <col_type> [INVISIBLE | VISIBLE] [<ai_shared_default>] [<on_update>] [<col_constraint>]

CREATE 문에서 type 바로 뒤에 visibility를 설정할 수 있도록 함.
- INDEX visibility에 사용되던 csql_grammar.y:opt_invisible 을 VISIBLE 용어 사용이 가능하도록 수정함.
ALTER 와 같이 컬럼 내용을 변경하는 명령에 대해 visibility를 수정할 수 있도록 함.
SHOW COLUMNS / DESC 명령에서 표시되는 테이블의 'Extra' 컬럼에 'INVISIBLE' 용어가 추가로 표시될 수 있도록 함.
Object Representation 의 flags 한 개 비트를 invisible로 사용.
- SM_ATTRIBUTE_FLAG에 SM_ATTFLAG_INVISIBLE_COLUMN 추가.
시스템 카탈로그 테이블 _db_attribute에 flags 컬럼을 추가함.
- flags 컬럼은 int형으로 비트값으로 해석하도록 함.
- Constraint가 아닌 옵션들 - 0번 비트부터 { Auto Increment, Invisible Column, Partitioning Key } 을 표시.
- 해당 비트값은 DB_ATTRIBUTE_OPTION_TYPE 에서 확인 가능.
시스템 카탈로그 뷰 db_attribute에 is_invisible 컬럼을 추가함.
- 뷰 이기에 flag를 풀어서 볼 수 있도록 컬럼을 추가하여 'YES' or 'NO' 로 표시되도록 함.

Remarks

[0a29901] : CREATE, ALTER에 visibility 관련 문법 추가. 기존에 사용되던 opt_invisible에 VISIBLE 용어 확장. SHOW COLUMNS 구문으로 'Extra' 컬럼에서 invisible 값 확인 가능.
error msg를 추가하였으나 경로가 없는 msg라 생각되어 삭제 예정

[1fa31f3] : 시스템 카탈로그 테이블, 뷰에서 flags 및 is_invisible 컬럼 추가하여 값 확인 가능.

[c6110b8] : 시스템 카탈로그 테이블, 뷰에서 flags 정보가 2048부터 시작하는 대신 1부터 시작하도록 수정.

…TE, ALTER). can check visibility using SHOW [FULL] COLUMNS or DESC. no visibility functioning now.

…ttribute)

src/base/object_representation_sr.c

src/storage/storage_common.h

src/storage/catalog_class.c

src/object/schema_system_catalog_install_query_spec.cpp

Co-authored-by: kangmin <[email protected]>

src/query/query_executor.c

src/compat/db_info.c

src/query/query_executor.c

Co-authored-by: jongmin-won <[email protected]>

Co-authored-by: kangmin <[email protected]>

…tribute's flags column

ctshim · 2025-12-08T02:20:34Z

@childyouth
이 PR이 [CBRD-26401] 전체를 커버하는 것인가요? 그중 일부만 하는 것이면 하위 이슈를 생성해서 연결해 주면 좋겠습니다.

childyouth · 2025-12-08T03:05:28Z

@childyouth 이 PR이 [CBRD-26401] 전체를 커버하는 것인가요? 그중 일부만 하는 것이면 하위 이슈를 생성해서 연결해 주면 좋겠습니다.

jira에 페이즈 단위로 하위 이슈를 생성하고 연결하겠습니다. 감사합니다.

ctshim · 2025-12-08T04:06:03Z

shared 또는 class attribute의 경우에도 invisible 지정이 필요할까요?

ctshim · 2025-12-08T05:02:53Z

src/query/query_executor.c

+
+	      /* add whitespace character if saved is not an empty string */
+	      const char *is_invisible_string = "invisible";
+	      size_t str_len = len + strlen (is_invisible_string) + 1;


strlen (is_invisible_string)의 값은 변하지 않는 값이므로 한번만 구할 수 있도록 변경해 두면 좋겠습니다.
static const invisible_str_length = strlen("invisible");와 같이 처리해 주는 것을 검토해 봐 주세요

현재 함수인 qexec_execute_build_columns 내부에 이러한 방식으로 strlen 하는 경우가 많이 있습니다.

아마도 해당 함수를 부르는 DESC / SHOW [FULL] COLUMNS 명령어가 자주 사용될 일이 없다보니 계산 코스트보단 메모리 사용을 없애려 한 것은 아닐까 생각됩니다.

해당 함수내에서 모두 static size_t로 변경을 진행해도 괜찮을까요?

childyouth · 2025-12-08T05:15:43Z

shared 또는 class attribute의 경우에도 invisible 지정이 필요할까요?

shared 는 alter가 불가능하여 invisible을 지원하지 않는 편이 좋겠다고 말씀해 주신 부분에 대해 동의하며 invisible은 일반 attribute에서만 지원하도록 하겠습니다.

class attribute에 대한 이해가 부족하여 상속 혹은 포인터 같은 형태의 어트리뷰트로 이해하고 있었습니다.
더 찾아본 결과 static 느낌으로 한 개의 레코드만 존재하는 특수 어트리뷰트이고, select * 로는 검색할 수 없는 타입이기에 invisible이 필요없다고 판단됩니다.

-> shared 와 class attribute 둘 다 invisible을 지원하지 않는 방향으로 수정하겠습니다. 감사합니다.

src/storage/catalog_class.c

src/parser/csql_grammar.y

src/query/query_executor.c

airnet73 · 2025-12-09T07:41:26Z

src/object/object_printer.cpp

      m_buf (" NOT NULL");
    }

+  // If the hidden column flag - 'system added invisible column' - is added to attribute.flags, the hidden column flag must be checked first.


주석 내용에 hidden column와 invisible column 두가지 용어가 사용되고 있습니다. 의도한 용어 인지요?

네, 오라클에선 system이 생성한 invisible column을 hidden column으로 지칭하고 있습니다. 기능적으로 hidden column은 invisible -> visible 설정이 불가능하게 되어있어 구분되어있습니다.

… CBRD-26401

childyouth · 2025-12-22T05:35:28Z

/run all

childyouth · 2025-12-23T07:15:03Z

/run shell

childyouth added 3 commits December 2, 2025 13:39

add invisible column to grammar (can set invisible/visible using CREA…

0a29901

…TE, ALTER). can check visibility using SHOW [FULL] COLUMNS or DESC. no visibility functioning now.

show invisible option or flags on system catalog (db_attribute, _db_a…

1fa31f3

…ttribute)

show flags on _db_attribute start at first bit

c6110b8

childyouth requested review from airnet73, ctshim, hyunikn, jongmin-won and kangmin5505 December 3, 2025 08:28

childyouth self-assigned this Dec 3, 2025

childyouth requested review from beyondykk9, hornetmj and shparkcubrid as code owners December 3, 2025 08:28

kangmin5505 reviewed Dec 4, 2025

View reviewed changes

Apply suggestion from @kangmin5505

526f296

Co-authored-by: kangmin <[email protected]>

jongmin-won reviewed Dec 4, 2025

View reviewed changes

src/query/query_executor.c Outdated Show resolved Hide resolved

src/compat/db_info.c Outdated Show resolved Hide resolved

jongmin-won reviewed Dec 4, 2025

View reviewed changes

src/query/query_executor.c Outdated Show resolved Hide resolved

childyouth and others added 5 commits December 4, 2025 12:23

Update src/query/query_executor.c

669b88d

Co-authored-by: jongmin-won <[email protected]>

Apply suggestions from code review

bbb6c91

Co-authored-by: kangmin <[email protected]>

function for system catalog '_db_attribute' flags attribute

52fe32a

apply codestyle. apply pull request comments. new function for _db_at…

ce465d4

…tribute's flags column

Merge branch 'feature/invisible_column' into CBRD-26401

03e7707

childyouth force-pushed the CBRD-26401 branch from 8eac52a to 03e7707 Compare December 4, 2025 08:14

ctshim reviewed Dec 8, 2025

View reviewed changes

src/storage/catalog_class.c Show resolved Hide resolved

ctshim reviewed Dec 8, 2025

View reviewed changes

src/storage/catalog_class.c Outdated Show resolved Hide resolved

childyouth marked this pull request as draft December 8, 2025 08:38

hyunikn reviewed Dec 9, 2025

View reviewed changes

src/parser/csql_grammar.y Outdated Show resolved Hide resolved

airnet73 reviewed Dec 9, 2025

View reviewed changes

src/query/query_executor.c Outdated Show resolved Hide resolved

airnet73 reviewed Dec 9, 2025

View reviewed changes

childyouth added 2 commits December 10, 2025 18:37

Apply suggestions from code review

1ed0c46

apply suggestions from code review

330ed7b

childyouth force-pushed the CBRD-26401 branch from aad2cf5 to 330ed7b Compare December 19, 2025 05:17

Merge remote-tracking branch 'upstream/feature/invisible_column' into…

94db153

… CBRD-26401

childyouth changed the title ~~[CBRD-26401] (Phase-1) Create/Alter invisible column~~ [CBRD-26447] (Phase-1) Create/Alter invisible column Dec 19, 2025

db_attribute use _db_attribute's flags with DB_ATTOPT_INVISIBLE_COLUMN

02d09bb

childyouth marked this pull request as ready for review December 19, 2025 07:11

childyouth requested review from airnet73, ctshim, hyunikn, jongmin-won and kangmin5505 December 19, 2025 08:28

shparkcubrid removed their request for review December 20, 2025 06:19

beyondykk9 approved these changes Dec 22, 2025

View reviewed changes

jongmin-won approved these changes Dec 24, 2025

View reviewed changes

[CBRD-26447] (Phase-1) Create/Alter invisible column #6687

Are you sure you want to change the base?

[CBRD-26447] (Phase-1) Create/Alter invisible column #6687

Uh oh!

Conversation

childyouth commented Dec 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose

INVISIBLE COLUMN

Phase 1

Implementation

Remarks

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ctshim commented Dec 8, 2025

Uh oh!

childyouth commented Dec 8, 2025

Uh oh!

ctshim commented Dec 8, 2025

Uh oh!

ctshim Dec 8, 2025

Choose a reason for hiding this comment

Uh oh!

childyouth Dec 8, 2025

Choose a reason for hiding this comment

Uh oh!

childyouth commented Dec 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

airnet73 Dec 9, 2025

Choose a reason for hiding this comment

Uh oh!

childyouth Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

childyouth commented Dec 22, 2025

Uh oh!

childyouth commented Dec 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

childyouth commented Dec 3, 2025 •

edited

Loading

childyouth commented Dec 8, 2025 •

edited

Loading