You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@celeborn.apache.org by "HolyLow (via GitHub)" <gi...@apache.org> on 2025/02/12 10:22:20 UTC
[PR] [CELEBORN-1863][CIP-14] Add TransportClient to cppClient [celeborn]
HolyLow opened a new pull request, #3097:
URL: https://github.com/apache/celeborn/pull/3097
### What changes were proposed in this pull request?
This PR adds TransportClient to cppClient.
### Why are the changes needed?
The TransportClient is responsible for sending/receiving the message to/from network.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Compilation and UTs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1863][CIP-14] Add TransportClient to cppClient [celeborn]
Posted by "FMX (via GitHub)" <gi...@apache.org>.
FMX commented on PR #3097:
URL: https://github.com/apache/celeborn/pull/3097#issuecomment-2660701055
> @FMX The CI procedure could be added in next commit. I am not familiar with github's CI construction, so maybe you could give me some guidance on this.
>
> I might need help especially on how to use the prebuilt container in CI, as the environment setup for cpp is tedious and should not run each time CI is triggered.
No problem. Let's add the CI in the next PR. I'll merge this one first.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1863][CIP-14] Add TransportClient to cppClient [celeborn]
Posted by "FMX (via GitHub)" <gi...@apache.org>.
FMX closed pull request #3097: [CELEBORN-1863][CIP-14] Add TransportClient to cppClient
URL: https://github.com/apache/celeborn/pull/3097
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1863][CIP-14] Add TransportClient to cppClient [celeborn]
Posted by "HolyLow (via GitHub)" <gi...@apache.org>.
HolyLow commented on PR #3097:
URL: https://github.com/apache/celeborn/pull/3097#issuecomment-2653288392
@FMX @RexXiong Could you kindly help review this PR? Thanks a lot.
Any suggestion is welcome.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1863][CIP-14] Add TransportClient to cppClient [celeborn]
Posted by "HolyLow (via GitHub)" <gi...@apache.org>.
HolyLow commented on PR #3097:
URL: https://github.com/apache/celeborn/pull/3097#issuecomment-2658373079
@FMX The CI procedure could be added in next commit. I am not familiar with github's CI construction, so maybe you could give me some guidance on this.
I might need help especially on how to use the prebuilt container in CI, as the environment setup for cpp is tedious and should not run each time CI is triggered.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org