tbonelee commented on PR #4893: URL: https://github.com/apache/zeppelin/pull/4893#issuecomment-2466045338
@Reamer I considered both `ZEPPELIN_EVENT_SERVER_PORT` and `ZEPPELIN_SERVER_RPC_PORT`. I chose the former because the term "SERVER" in the latter could be too general, making it unclear which server it refers to, and `ZEPPELIN_SERVER_RPC_PORT` may be easily confused with `ZEPPELIN_SERVER_RPC_PORTRANGE` due to the similarity in names. However, aligning with previous variable names might improve predictability as well, which is why I didn't have a strong preference for the former option from the beginning. What do you think? -- 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: reviews-unsubscr...@zeppelin.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
