codope commented on code in PR #6715:
URL: https://github.com/apache/hudi/pull/6715#discussion_r975314604


##########
hudi-common/src/main/resources/hbase-site.xml:
##########
@@ -1699,13 +1699,16 @@ possible configurations would overwhelm and obscure the 
important.
       Implementation of the status publication with a multicast message.
     </description>
   </property>
-  <property>
-    <name>hbase.status.listener.class</name>
-    
<value>org.apache.hadoop.hbase.client.ClusterStatusListener$MulticastListener</value>
-    <description>
-      Implementation of the status listener with a multicast message.
-    </description>
-  </property>
+
+  <!-- Hbase related classes used in hudi bundle packages are in shaded 
pattern,
+  this config can cause ClassNotFoundException -->
+  <!--<property>-->
+  <!--  <name>hbase.status.listener.class</name>-->
+  <!--  
<value>org.apache.hadoop.hbase.client.ClusterStatusListener$MulticastListener</value>-->
+  <!--  <description>-->
+  <!--    Implementation of the status listener with a multicast message.-->
+  <!--  </description>-->
+  <!--</property>-->

Review Comment:
   So, if I understand correctly, this property is being removed here because 
you want your override to take effect? If so, just remove it instead of 
commenting out.



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