iwasakims commented on code in PR #1043:
URL: https://github.com/apache/bigtop/pull/1043#discussion_r999157622
##########
bigtop_toolchain/manifests/installer.pp:
##########
@@ -14,6 +14,7 @@
# limitations under the License.
class bigtop_toolchain::installer {
+ include bigtop_toolchain::jdk11
include bigtop_toolchain::jdk
Review Comment:
Hmm. /usr/bin/java points to Java 11 after installing both OpenJDK 8 and
OpenJDK 11 on Ubuntu. If we want to retain existing setup in which OpenJDK 8 is
preferred as default, the alternatives should be switched back (by running
update-java-alternatives?).
```
$ update-alternatives --display java
java - auto mode
link best version is /usr/lib/jvm/java-11-openjdk-amd64/bin/java
link currently points to /usr/lib/jvm/java-11-openjdk-amd64/bin/java
link java is /usr/bin/java
slave java.1.gz is /usr/share/man/man1/java.1.gz
/usr/lib/jvm/java-11-openjdk-amd64/bin/java - priority 1111
slave java.1.gz: /usr/lib/jvm/java-11-openjdk-amd64/man/man1/java.1.gz
/usr/lib/jvm/java-8-openjdk-amd64/jre/bin/java - priority 1081
slave java.1.gz: /usr/lib/jvm/java-8-openjdk-amd64/jre/man/man1/java.1.gz
$ update-java-alternatives -l
java-1.11.0-openjdk-amd64 1111
/usr/lib/jvm/java-1.11.0-openjdk-amd64
java-1.8.0-openjdk-amd64 1081
/usr/lib/jvm/java-1.8.0-openjdk-amd64
```
--
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]
