Apache9 commented on a change in pull request #3992:
URL: https://github.com/apache/hbase/pull/3992#discussion_r778511208



##########
File path: src/main/asciidoc/_chapters/configuration.adoc
##########
@@ -289,14 +289,14 @@ Use the following legend to interpret this table:
 link:https://hadoop.apache.org/cve_list.html[CVEs] so we drop the support in 
newer minor releases
 * icon:exclamation-circle[role="yellow"] = Not tested, may/may-not function
 
-[cols="1,2*^.^", options="header"]
+[cols="1,3*^.^", options="header"]
 |===
-| | HBase-1.7.x | HBase-2.3.x
-|Hadoop-2.10.x | icon:check-circle[role="green"] | 
icon:check-circle[role="green"]
-|Hadoop-3.1.0 | icon:times-circle[role="red"] | icon:times-circle[role="red"]
-|Hadoop-3.1.1+ | icon:times-circle[role="red"] | 
icon:check-circle[role="green"]
-|Hadoop-3.2.x | icon:times-circle[role="red"] | icon:check-circle[role="green"]
-|Hadoop-3.3.x | icon:times-circle[role="red"] | icon:check-circle[role="green"]
+| | HBase-1.7.x | HBase-2.3.x | HBase-2.4.x

Review comment:
       For me since 2.3.x is EOL while 2.4.x is still in maintanence, I prefer 
we make a separated column for 2.4.x. We can merge the columns for EOL release 
lines as they will not be changed in the future.
   
   And another thing is that in the past we will remove the EOL release lines 
soon in this matrix, both for HBase and Hadoop, but for me I think we'd better 
keep it for some more time as lots of our users are still on the old releases, 
for example, my company still needs to support hadoop-2.6.0 for our customers...
   Anyway, this could a another issue.




-- 
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]


Reply via email to