dubbo icon indicating copy to clipboard operation
dubbo copied to clipboard

[Feature] triple protocol use netty ProtobufEncoder and ProtobufDecoder to replace .proto

Open JunJieLiu51520 opened this issue 1 year ago • 4 comments

Pre-check

  • [X] I am sure that all the content I provide is in English.

Search before asking

  • [X] I had searched in the issues and found no similar feature requirement.

Apache Dubbo Component

Java SDK (apache/dubbo)

Descriptions

and other way, this can decrease the compiler cost, current sdk repository source is too heavely

Related issues

No response

Are you willing to submit a pull request to fix on your own?

  • [X] Yes I am willing to submit a pull request on my own!

Code of Conduct

JunJieLiu51520 avatar Sep 24 '24 15:09 JunJieLiu51520

@oxsean do you agree this ?

JunJieLiu51520 avatar Sep 24 '24 15:09 JunJieLiu51520

Sorry, I didn't quite understand what want to do, can you add more info?

oxsean avatar Sep 25 '24 02:09 oxsean

image

i say that if we add a protobuf channelhandler whether mean we need not define .proto file and add (de)serialize logic,

i think it's heavly in dubbo-rpc-triple module, right ?

another reason is dubbo sdk build getting slower and slower and when sync from dubbo to my reponsity always compiler failed

JunJieLiu51520 avatar Sep 25 '24 11:09 JunJieLiu51520

The issue is not on the same level. The ability to encode and decode also relies on having classes generated based on proto. Moreover, Triple is an RPC framework, and its data packets are not in raw proto format.

oxsean avatar Sep 27 '24 07:09 oxsean