Skip to content

Conversation

@jj1kim
Copy link

@jj1kim jj1kim commented Feb 24, 2025

GRPC를 지원하기 위한 기능 개발입니다.

protobuf 코드의 경우 같은 와플스튜디오 그룹 내 hodu-proto 레포지토리에 있으며, 이를 서브모듈로 끌어오는 방식입니다.

(본 설명 추후 수정 및 보완 예정)

@jj1kim jj1kim requested a review from minkyu97 February 24, 2025 08:33
@woohm402 woohm402 assigned jj1kim and unassigned woohm402 Feb 24, 2025
@woohm402 woohm402 requested review from minkyu97 and woohm402 and removed request for minkyu97 February 24, 2025 10:59
@minkyu97
Copy link
Collaborator

실제 GRPC를 사용하는 부분은 별도 PR로 올리실 예정인가요? 아니면 이 PR에 이어서 작성하나요??

@jj1kim
Copy link
Author

jj1kim commented Feb 24, 2025

실제 GRPC를 사용하는 부분은 별도 PR로 올리실 예정인가요? 아니면 이 PR에 이어서 작성하나요??

아 이어서 작성 중입니다!! 그 말을 안 적었네요... protobuf의 부족한 serializing을 보완해서 원래 http랑 메시지 내용이 동일하게 맞추는 코드 먼저 추가하고, 실제 grpc 서비스 만드는 부분도 곧 추가할 예정입니다.

@minkyu97
Copy link
Collaborator

minkyu97 commented Feb 24, 2025

[hodu-proto 피드백]
P2) 공식문서를 참고해서, 모든 필드에 optional을 붙이는 게 좋아보입니다.

Q) 제가 잘 모르는 것일 수도 있는데요, protobuf의 메시지 명이랑 실제 사용 중인 클래스 명이랑 겹쳐도 괜찮은 건가요?? 아니면 둘을 통합해주는 라이브러리가 있나요? 필요하다면 protobuf 메시지/enum에는 prefix나 postfix를 추가해야할 수도 있을 거 같아서요
Q) remote type pattern 이 뭔가요?

@jj1kim
Copy link
Author

jj1kim commented Feb 24, 2025

[hodu-proto 피드백] P2) 공식문서를 참고해서, 모든 필드에 optional을 붙이는 게 좋아보입니다.

넵 repeated가 없는 필드는 optional을 붙여두겠습니다.

Q) 제가 잘 모르는 것일 수도 있는데요, protobuf의 메시지 명이랑 실제 사용 중인 클래스 명이랑 겹쳐도 괜찮은 건가요?? 아니면 둘을 통합해주는 라이브러리가 있나요? 필요하다면 protobuf 메시지/enum에는 prefix나 postfix를 추가해야할 수도 있을 거 같아서요

저는 모듈이 달라서 괜찮다고 일단 생각하긴 했습니다. protobuf 코드는 다른 곳에 있고 이걸 서브모듈로 끌어와서 빌드하는 것이니, prost가 protobuf 코드를 빌드한다고 하더라도 메인 코드와는 모듈이 달라서 문제가 없을 것이라고 생각했습니다. 관련해서 꽤 찾아보긴 했는데 명확한 답이 있는 것들이 없어서 생각한 대로 올려둔 상태입니다. 주시하고 있다가 만약 문제가 생기면 수정해도 괜찮을까요?

Q) remote type pattern 이 뭔가요?

아 이거 공식 용어인 줄 알았는데 아니었네요;; 제가 하고 싶었던 말은 serde에서 "remote="를 써서 다른 크레이트에서 이미 정의된 구조체를 끌어와서 수정하는 방식이었습니다.

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.

4 participants