xumanbu commented on issue #1373: URL: https://github.com/apache/incubator-uniffle/issues/1373#issuecomment-2325555937
> @xumanbu This is a great feature! But it seems not finished. I guess it lack a piece about how server reject the ` large partition` and split to write to another server. > > The existing logic handled `send data failed` and reassign the partitions related to these server, but for `large partition` case, the server is health, so the `send data` request should be succeed in general but failed for few partition data, so we should handle this case also. > > WDYT? Good consideration scene. But I think it's Ok at now. The `send data` request may fail for large partitions due to the server rejecting it with `StatusCode.NO_BUFFER_FOR_HUGE_PARTITION`. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
