Skip to content

[CBRD-26527] drop table시 사용된 페이지가 반환되지 오류 - 히든 유틸리티 지원#6883

Merged
hornetmj merged 18 commits intoCUBRID:release/11.3from
hornetmj:CBRD-26527_drop_v11.3
Apr 2, 2026
Merged

[CBRD-26527] drop table시 사용된 페이지가 반환되지 오류 - 히든 유틸리티 지원#6883
hornetmj merged 18 commits intoCUBRID:release/11.3from
hornetmj:CBRD-26527_drop_v11.3

Conversation

@hornetmj
Copy link
Copy Markdown
Contributor

@hornetmj hornetmj commented Mar 2, 2026

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

Purpose

cleanfiledb 히든 유틸리티 제공

$ cubrid cleanfiledb
cleanfiledb: Delete invalid files and reclaim storage space.
usage: cubrid cleanfiledb [OPTION] database-name

valid options:
    -S, --SA-mode                 stand-alone mode execution
    -C, --CS-mode                 client-server mode execution
    -o, --output-file=FILE        redirect output message to FILE; default: none.
    -l, --list                    list all files
    -c, --clean-invalid-file      clean invalid files

Implementation

N/A

Remarks

N/A

@hornetmj hornetmj requested a review from mhoh3963 March 2, 2026 11:25
@hornetmj hornetmj marked this pull request as ready for review March 2, 2026 11:29
Comment thread msg/en_US/msg.txt Outdated
Comment thread src/executables/util_cs.c Outdated
Comment thread src/executables/util_cs.c Outdated
Comment thread src/executables/util_cs.c
Comment thread src/storage/file_manager.c Outdated
Comment thread src/storage/file_manager.c Outdated
Comment thread src/storage/file_manager.c
Comment thread src/storage/file_manager.c Outdated
Comment thread src/storage/file_manager.c Outdated
Comment thread src/storage/file_manager.c Outdated
Comment thread msg/en_US.utf8/utils.msg Outdated
return;
}

outfp = tmpfile ();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

기술팀에서 /tmp에 권한 문제로부터 오류를 방지하기 위해 CUBRID_TMP 환경 변수를 사용하는데, tmpfile()는 NULL을 반환해서 해당 함수가 오류를 발생하게 되는데, 원인을 찾기 힘들 것 같습니다.
tmpfile()를 사용하는 곳이 많아, 이번 PR에서는 수정하지 말고, 다른 이슈로 수정 필요해보이네요.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 31, 2026

Reviews (2): Last reviewed commit: "2nd review feedback" | Re-trigger Greptile

Comment thread src/storage/file_manager.c Outdated
Comment thread src/communication/network_interface_cl.c
Comment thread src/communication/network_interface_sr.c
Comment thread src/storage/file_manager.c
Comment thread src/storage/file_manager.c
Comment thread src/storage/file_manager.c Outdated
@hornetmj hornetmj requested a review from mhoh3963 April 1, 2026 04:08
Comment thread src/communication/network_interface_sr.c
Comment thread src/executables/util_cs.c Outdated
@hornetmj hornetmj force-pushed the CBRD-26527_drop_v11.3 branch from beedb52 to 9e7061e Compare April 1, 2026 05:26
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 1, 2026

Reviews (3): Last reviewed commit: "greptile review" | Re-trigger Greptile

Comment thread src/storage/file_manager.c Outdated
Comment thread src/storage/file_manager.c
Comment thread msg/en_US/utils.msg
Comment thread src/executables/util_cs.c
Comment thread src/executables/util_cs.c Outdated
Comment thread src/communication/network_common.cpp Outdated
Copy link
Copy Markdown
Contributor

@mhoh3963 mhoh3963 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hornetmj
11.3 이하 msg 파일 반영을 제외하고 꼭 수정할 필요하지는 않지만 참고해주세요.
그리고 greptile이 review 해 주는 내용은 확인해봐주세요.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 1, 2026

Reviews (4): Last reviewed commit: "review feedback" | Re-trigger Greptile

Comment thread src/storage/file_manager.c Outdated
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 1, 2026

Reviews (5): Last reviewed commit: "greptile review" | Re-trigger Greptile

@hornetmj hornetmj merged commit 1290a67 into CUBRID:release/11.3 Apr 2, 2026
9 checks passed
InChiJun added a commit to InChiJun/cubrid that referenced this pull request Apr 17, 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.

2 participants