Hi All, Thanks for your comments. I agree that TransportMessages should be divided into several modules to enhance readability and extensibility. However, we also need to consider the compatibility issues associated with this change. Could we start by merging Resource into TransportMessages as an initial step?
I updated the proposal and please help review again. Thanks, He Jiaming Xie <jiaming.x...@gmail.com> 于2024年12月20日周五 22:58写道： > > Hi He, > Thanks for the proposal. > > It seems that the proposal involves refactoring TransportMessages.proto, > and the current design mainly focuses on the compatibility problem in Java > code. Yet currently the CppClient proposed in CIP-14[1] is under active > development and already has dependency on Java side's transport proto file[2]. > To make the message compatible, the CppClient has to use the same proto > definition as well as the same protobuf version to Java side. > > Although CppClient is still on its early stage, I think maybe give a look on > how the CppClient side should adapt to the refactored proto layout would > make the proposal more complete. > > [1] > https://cwiki.apache.org/confluence/display/CELEBORN/CIP-14+Support+CppClient+in+Celeborn > [2] > https://github.com/apache/celeborn/commit/80ebb19836f886451b0115e8b0eaa53dac3d94da > > Regards, > Jiaming > > > he zhao <zhaohehe0...@gmail.com> 于2024年12月17日周二 16:57写道： > > > > Hi all, > > > > I would like to start a discussion on CIP-16: Merge resource proto and > > transport proto[1]. > > > > Looking forward to everyone's feedback and suggestions. Thanks in advance. > > > > [1] > > https://docs.google.com/document/d/1_sgDfyizkvQ7PFryIH_AbD320fxGvFcLCb_EZFNs7Nc/edit?usp=sharing > > > > Regards, > > He
