virajjasani commented on a change in pull request #1871:
URL: https://github.com/apache/hbase/pull/1871#discussion_r436954122
##########
File path: src/main/asciidoc/_chapters/developer.adoc
##########
@@ -420,6 +419,27 @@ Tests may not all pass so you may need to pass
`-DskipTests` unless you are incl
To pick a particular Hadoop 3.y release, you'd set hadoop-three.version
property e.g. `-Dhadoop-three.version=3.0.0`.
+[[maven.build.jdk11_hadoop3]]
+==== Building with JDK11 and Hadoop3
+
+The following guidance applies to HBase versions [2.3, 3), where both JDK8 and
JDK11 are supported,
+and both Hadoop2 and Hadoop3 are supported.
+
+HBase manages JDK-specific build settings using maven profiles. The profile
appropriate to the JDK
+in use is automatically selected. JDK8 supports both Hadoop2 and Hadoop3. For
JDK11, only Hadoop3
+is supported, which means you must specify `-Dhadoop.profile=3.0` when
building with JDK11.
Review comment:
By specifying `-Dhadoop.profile=3.0`, users are consistent across all
JDK 11 supported branches. For instance, branch-2 has `hadoop-2.0` active by
default I believe. Hence, with this doc, users can try `-Dhadoop.profile=3.0`
for all supported branches and there is no harm with master branch also. This
should be fine. Thought?
##########
File path: src/main/asciidoc/_chapters/developer.adoc
##########
@@ -420,6 +419,27 @@ Tests may not all pass so you may need to pass
`-DskipTests` unless you are incl
To pick a particular Hadoop 3.y release, you'd set hadoop-three.version
property e.g. `-Dhadoop-three.version=3.0.0`.
+[[maven.build.jdk11_hadoop3]]
+==== Building with JDK11 and Hadoop3
+
+The following guidance applies to HBase versions [2.3, 3), where both JDK8 and
JDK11 are supported,
+and both Hadoop2 and Hadoop3 are supported.
+
+HBase manages JDK-specific build settings using maven profiles. The profile
appropriate to the JDK
+in use is automatically selected. JDK8 supports both Hadoop2 and Hadoop3. For
JDK11, only Hadoop3
+is supported, which means you must specify `-Dhadoop.profile=3.0` when
building with JDK11.
Review comment:
Oh okk, got it. Thanks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
