You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/15 10:01:45 UTC

[GitHub] [pulsar] asafm opened a new pull request, #17669: [fix][doc] Fix M1 JVM Installation Instructions

asafm opened a new pull request, #17669:
URL: https://github.com/apache/pulsar/pull/17669

   ### Modifications
   
   Fix the documentation of installing JVM on M1 - remove one installation method of sdkman (using brew) and add the deleted instructions to building Pulsar on README
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc-required` 
   (Your PR needs to update docs and you will update later)
   
   - [ ] `doc-not-needed` 
   (Please explain why)
   
   - [x] `doc` 
   (Your PR contains doc changes)
   
   - [ ] `doc-complete`
   (Docs have been already added)
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973689315


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   So the meaningful information here is, that if you fail to run Pulsar on M1 due to RocksDB JNI error (with a typical stack trace), the reason is that you need an X86 JDK for older versions. Then mention some methods to achieve it as you like.
   
   I get confused the first time: I'm an M1 user, and this section seems written for me. But it's not. It's for those who running Pulsar <= 2.10 on M1 and get the RocksDB JNI load exception.



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] asafm commented on pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
asafm commented on PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#issuecomment-1262621095

   @tisonkun How do we continue from here?


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] asafm commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
asafm commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973678845


##########
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:
   This issue happened to me and 5 other engineers I know of just as an example. 
   Our approach, especially on getting started should be giving you everything you need to get started - in the README case, it's building the product. M1 is notorious to have many quirks. Anything we can do to help will be greatly appreciated.
   
   Just mentioning a few people recently running into those issues @geomagilles @teabot @nahguam just to name a few
   
   Bear in mind, that they can ignore this recommendation. Most newcomers will appreciate it as much as I did.
   
   
   



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973688501


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   And I just notice that it misleads me because it's not about "Install JDK on M1" - it's about "Install X86 JDK on M1".
   
   https://www.google.com/search?q=Install+x86+JDK+on+M1 gives you:
   
   * https://itnext.io/how-to-install-x86-and-arm-jdks-on-the-mac-m1-apple-silicon-using-sdkman-872a5adc050d
   * https://github.com/sdkman/sdkman-cli/issues/1068
   
   That can help.



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973687982


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   This doesn't stand any more on the nightly version. See also https://github.com/apache/pulsar/issues/12166#issuecomment-1237601981.
   
   I agree that there can be a few troubleshooting to work on a different environment with older version, and I'd prefer a discussion thread or a mac specific section like [Hadoop does](https://github.com/apache/hadoop/blob/a73c4804d882d9bc1a424080f20c4902cb9db832/BUILDING.txt#L389-L418).



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973689315


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   So the meaningful information here is, that if you fail to run Pulsar on M1 due to RocksDB JNI error (with a typical stack trace), the reason is that you need an X86 JDK for older versions. Then mention some methods to achieve it as you like.



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] Anonymitaet merged pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
Anonymitaet merged PR #17669:
URL: https://github.com/apache/pulsar/pull/17669


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973688501


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   And I just notice that it misleads me because it's not about "Install JDK on M1" - it's about "Install X86 JDK on M1" (if you get a RocksDB JNI load exception).
   
   https://www.google.com/search?q=Install+x86+JDK+on+M1 gives you:
   
   * https://itnext.io/how-to-install-x86-and-arm-jdks-on-the-mac-m1-apple-silicon-using-sdkman-872a5adc050d
   * https://github.com/sdkman/sdkman-cli/issues/1068
   
   That can help.



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
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 image users have multiple approach to do this and we don't have to use forty lines for it. If someone asked, we resolve the discuss 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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
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 image users have multiple approach to do this and we don't have to use forty lines for it. If someone asked, we resolve the discuss 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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973687982


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   This doesn't stand any more on the nightly version. See also https://github.com/apache/pulsar/issues/12166#issuecomment-1237601981.
   
   I agree that there can be a few troubleshooting to work on a different environment with older version, and I'd prefer a discussion thread (https://github.com/apache/pulsar/issues/12166 seems a good place to comment these instructions and closed) or a mac specific section like [Hadoop does](https://github.com/apache/hadoop/blob/a73c4804d882d9bc1a424080f20c4902cb9db832/BUILDING.txt#L389-L418) (but still, on <= 2.10 branches?).



##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   This doesn't stand any more on the nightly version. See also https://github.com/apache/pulsar/issues/12166#issuecomment-1237601981.
   
   I agree that there can be a few troubleshooting to work on a different environment with older version, and I'd prefer a discussion thread (https://github.com/apache/pulsar/issues/12166 seems a good place to comment these instructions and closed).



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973687982


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   This doesn't stand any more on the nightly version. See also https://github.com/apache/pulsar/issues/12166#issuecomment-1237601981.
   
   I agree that there can be a few troubleshooting to work on a different environment with older version, and I'd prefer a discussion thread or a mac specific section like [Hadoop does](https://github.com/apache/hadoop/blob/a73c4804d882d9bc1a424080f20c4902cb9db832/BUILDING.txt#L389-L418) (but still, on < 2.10 branches?).



##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   This doesn't stand any more on the nightly version. See also https://github.com/apache/pulsar/issues/12166#issuecomment-1237601981.
   
   I agree that there can be a few troubleshooting to work on a different environment with older version, and I'd prefer a discussion thread or a mac specific section like [Hadoop does](https://github.com/apache/hadoop/blob/a73c4804d882d9bc1a424080f20c4902cb9db832/BUILDING.txt#L389-L418) (but still, on <= 2.10 branches?).



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r973690909


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   And perhaps after we release 2.11.0 and users start to use the new version, it will be less and less useful.
   
   > M1 is notorious to have many quirks. Anything we can do to help will be greatly appreciated.
   
   I agree. So +0 on this patch now. FYI - I encounter several other issues, and I prefer to resolve troubleshootings in discussions instead of a manual try to cover most of them:
   
   * https://github.com/apache/pulsar/issues/16639
   * https://github.com/apache/pulsar/issues/16652



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r971800739


##########
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:
   For example, I install JDK on M1 simply with `sdk install java` without all the troubleshooting here and it works well.



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] asafm commented on a diff in pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
asafm commented on code in PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#discussion_r975003904


##########
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
+In the current version, Pulsar uses a BookKeeper version which in turn uses RocksDB. RocksDB is compiled to work on x86 architecture and not ARM. Therefore, Pulsar can only work with x86 JDK. This is planned to be fixed in future versions of Pulsar.

Review Comment:
   Ok. I agree. It's mostly targeted at users trying to develop Pulsar and since it was solved on master, it's not beneficial anymore.
   



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] asafm commented on pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
asafm commented on PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#issuecomment-1251975641

   @tisonkun I reduced the PR by just fixing 2.10 instructions then


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on pull request #17669: [fix][doc] Fix M1 JVM Installation Instructions

Posted by GitBox <gi...@apache.org>.
tisonkun commented on PR #17669:
URL: https://github.com/apache/pulsar/pull/17669#issuecomment-1262625146

   Let me try to find a committer to help with reviewing and merging. I don't have write permission here :)
   
   @momo-jun @Anonymitaet @merlimat 


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org