Skip to content

[APIS-1088] Remove protocol compatibility - #83

Merged
hwany7seo merged 5 commits into
CUBRID:developfrom
hwany7seo:protocol_v9
Aug 10, 2026
Merged

[APIS-1088] Remove protocol compatibility#83
hwany7seo merged 5 commits into
CUBRID:developfrom
hwany7seo:protocol_v9

Conversation

@hwany7seo

Copy link
Copy Markdown
Collaborator

http://jira.cubrid.org/browse/APIS-1088

Purpose

V9 미만의 호환성 코드 제거

Implementation

  • 호환성 코드 제거
  • 버전 업데이트 (11.4.0)
  • V8이하 버전 접속 시 connect 차단 및 알림.

Remark

N/A

@hwany7seo
hwany7seo requested review from Kang-dot and Srltas July 27, 2026 07:56
@hwany7seo hwany7seo self-assigned this Jul 27, 2026
@greptile-apps

greptile-apps Bot commented Jul 27, 2026

Copy link
Copy Markdown

Reviews (1): Last reviewed commit: "- Update version to 11.4" | Re-trigger Greptile

Comment thread src/jdbc/cubrid/jdbc/jci/UClientSideConnection.java Outdated
@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown

Reviews (2): Last reviewed commit: "- apply code format" | Re-trigger Greptile

@Srltas Srltas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

더 이상 참조하는 곳이 없는 코드는 정리하는 것이 어떤가요?

대상 위치
PROTOCOL_V0 ~ PROTOCOL_V8 UConnection.java:91-99
protoVersionIsSame() UConnection.java:1883
UFunctionCode.CURSOR_CLOSE_FOR_PROTOCOL_V2 UFunctionCode.java:93
BrokerHandler.cancelBroker() BrokerHandler.java:289

public API를 바로 제거하는 것이 부담이 된다면 @Deprecated를 추가하는 것도 좋을 것 같습니다.

Comment on lines 428 to 434
@@ -435,25 +435,17 @@ private void connectDB(int timeout) throws IOException, UJciException {

@Srltas Srltas Jul 29, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

구 방식 프로토콜을 구분하는 부분으로 이제 더 이상 유지할 필요가 없을 것 같습니다.

버전을 계산 후 거부하는 것보다 해당 비트를 검사 후 구 버전이면 바로 거부하는 것으로 수정하는 것이 어떤가요?

해당 로직을 제거하면서 makeBrokerVersion(), BROKER_INFO_MAJOR_VERSION, BROKER_INFO_MINOR_VERSION, BROKER_INFO_PATCH_VERSION도 더 이상 사용하는 곳이 없어 정리하는 것도 좋아보입니다.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

대상 검토
PROTOCOL_V0 ~ PROTOCOL_V8 유지
protoVersionIsSame() protoVersionIsAbove를 제외하고 삭제
CURSOR_CLOSE_FOR_PROTOCOL_V2 유지
BrokerHandler.cancelBroker() cancelBrokerEx->cancelBroker로 변경
구 버전 관련사항 삭제
CTP 테스트를 고려하여 위와 같이 정리하였습니다.

@hwany7seo

Copy link
Copy Markdown
Collaborator Author

@greptile 리뷰 부탁해

@greptile-apps

greptile-apps Bot commented Aug 7, 2026

Copy link
Copy Markdown

Reviews (3): Last reviewed commit: "- removed about old broker version" | Re-trigger Greptile

@hwany7seo
hwany7seo requested a review from Srltas August 7, 2026 05:22
@hwany7seo
hwany7seo merged commit 2a9ca81 into CUBRID:develop Aug 10, 2026
4 checks passed
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