tisonkun commented on code in PR #17669: URL: https://github.com/apache/pulsar/pull/17669#discussion_r971799555
########## README.md: ########## @@ -138,6 +138,47 @@ Requirements: > Note: this project includes a [Maven > Wrapper](https://maven.apache.org/wrapper/) that can be used instead of a > system installed Maven. > Use it by replacing `mvn` by `./mvnw` on Linux and `mvnw.cmd` on Windows in > the commands below. +### Install JDK on M1 Review Comment: I wrote: > "how to install JDK on M1" is not suitable to document on the Get Started page. Imagine that you don't tell users how to buy a computer. It's far away from Pulsar topics. Users can resolve it with Google, StackOverflow, etc. in https://github.com/apache/pulsar/issues/17409#issuecomment-1236839244. This stands for README.md also. I can imagine users have multiple approaches to do this and we don't have to use forty lines for it. If someone asked, we resolve the discussion or issue. And followers should be able to search or we simply answer by throwing the link. -- 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]
