Skip to content

INTERNAL: Generate engine bindings from the arcus-memcached headers - #3

Merged
jhpark816 merged 1 commit into
developfrom
bindings
Oct 24, 2026
Merged

jhpark816 merged 1 commit into
developfrom
bindings

Conversation

@ing-eoking

@ing-eoking ing-eoking commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

🔗 Related Issue

⌨️ What I did

  • arcus-memcached 헤더(C)로 엔진 바인딩(Rust)을 생성하는 build.rs를 추가합니다.
    • 크레이트 루트의 config.h, config_static.h, include/memcached/*.h를 bindgen으로 변환해 OUT_DIR/engine_api.rs를 만듭니다.
    • -include config.h로 서버가 configure된 ENABLE_* 설정을 그대로 반영합니다.
    • layout test를 켜 두어, 생성된 구조체의 크기·정렬·오프셋이 헤더와 다르면 빌드가 실패합니다.
  • 헤더와 생성된 바인딩은 저장소에 현재는 커밋하지 않습니다. (이후 서버가 릴리즈가 되면 커밋할 예정)

Summary by CodeRabbit

  • 새 기능
    • Rust 애플리케이션에서 memcached 엔진과 연동할 수 있는 인터페이스를 제공합니다.
    • 엔진 헤더를 바탕으로 빌드 시 인터페이스 정보를 생성합니다. 필요한 헤더나 설정 파일이 없거나 정보를 생성하지 못하면, 준비에 필요한 안내와 함께 오류를 표시합니다.

@coderabbitai

This comment was marked as resolved.

@ing-eoking
ing-eoking requested a review from zhy2on October 23, 2026 11:50
coderabbitai[bot]

This comment was marked as resolved.

@jhpark816

Copy link
Copy Markdown

@ing-eoking 최신 코드 기준으로 rebase 바랍니다.

@ing-eoking

Copy link
Copy Markdown
Collaborator Author

@jhpark816

rebase 되었습니다.

@ing-eoking
ing-eoking requested a review from jhpark816 October 24, 2026 06:38
@ing-eoking
ing-eoking marked this pull request as draft October 24, 2026 06:48
@ing-eoking
ing-eoking marked this pull request as ready for review October 24, 2026 08:04
@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@ing-eoking

ing-eoking commented Oct 24, 2026 •

Copy link
Copy Markdown
Collaborator Author

@jhpark816 @namsic @zhy2on

CI 관련 일부 수정이 필요했고, CodeRabbit(AI 코드 리뷰 툴)에서 지적한 사항이
아직 해결되지 않아 잠시 Draft 상태로 변경했었습니다.

CodeRabbit에서 지적한 내용은 include/memcached 내부 헤더 파일이 일부 누락된 경우에 나오는 메시지와
config.h, config_static.h, include/memcached 누락 시 출력되는 메시지를 동일하게 맞추라는 것이었습니다.

현재는 누락 시 출력할 메시지를 COPY_HINT로 정의하고, 두 경우 모두 해당 메시지를 출력하도록 수정했습니다.

변경이 끝났으므로 리뷰 진행해 주시면 감사하겠습니다.

@jhpark816

Copy link
Copy Markdown

@namsic @zhy2on
먼저 merge해 둡니다.
나중에라도 확인하고 이슈가 있으면 알려주세요.

@jhpark816
jhpark816 merged commit 0c39580 into develop Oct 24, 2026
4 checks passed
@zhy2on

zhy2on commented Oct 24, 2026

Copy link
Copy Markdown
Member

이상 없습니다

@ing-eoking
ing-eoking deleted the bindings branch October 24, 2026 12:12

This branch was previously deployed

1 inactive deployment
ee — 30565563 Deployed Oct 24, 2026 by ing-eoking via Build arcus-memcached #18
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.

3 participants