william-wang commented on pull request #35422: URL: https://github.com/apache/spark/pull/35422#issuecomment-1032207260
> Thanks. You can make separate PRs, but you cannot make an intermediate namespace `min` here according to your plan. You need to change like `min.cpu` -> `minCpu`, `min.memory` -> `minMemory`. In addition, `member` looks too general to be used in that context. Please revise into a more meaningful name. @dongjoon-hyun Thanks for your comments. It's reasonable to change `min.cpu` -> `minCPU`. How about change `min.member -> minMember` which keeps the same style with `minCPU`. It is more easier to understand. -- 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]
