jerqi commented on PR #2175: URL: https://github.com/apache/incubator-uniffle/pull/2175#issuecomment-2421514742
> @jerqi Reference to others > > * Hadoop `"ipc.server.handler.queue.size"` default to `100`, it is too small for nowaday. > * Alluxio default to max pool size, which default to 500. > > I have no idea to choose one of them > > 1. Default to MAX Integer, so it can have not effect to existing code, but really, it could make server crash for huge current scenario, but we can set it to a proper value. > 2. Default to max pool size which default to 2 * rpcExecutorSize = 2000, base on this default config, server could accept and hold 4000 rpc call, which could make `reject` rpc call, so we need to enlarge the rpcExecutorSize default from 1000 to 5000, so that we can accept and hold 10000 rpc call, it could be a little better. > 3. Default to a constant value, let's say 10000, it could be enough for these few year. > > Think twice, it really hard to choose, but i could choose choice 1 if I have to choose one. > > How about your idea? Ok for me. -- 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]
