NihalJain commented on code in PR #6864:
URL: https://github.com/apache/hbase/pull/6864#discussion_r2016358337


##########
hbase-rest/pom.xml:
##########
@@ -279,7 +279,7 @@
           <skipAssembly>true</skipAssembly>
         </configuration>
       </plugin>
-      <!-- Copy the static web resources (JS, CSS, images, etc.) from 
hbase-server sources -->
+      <!-- Copy the static web resources (JS, CSS, images, etc.) from 
hbase-server -->

Review Comment:
   Hi David any reason to move js inside the main jar assembly back again? I 
think this regresses requirement 
[HBASE-28921](https://issues.apache.org/jira/browse/HBASE-28921) [Avoid 
bundling hbase-webapps folder in default jars 
(](https://github.com/apache/hbase/commit/16c51d83ac980495b18d06e0b7480c665c99275b)https://github.com/apache/hbase/pull/6388[)](https://github.com/apache/hbase/commit/16c51d83ac980495b18d06e0b7480c665c99275b)



-- 
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: issues-unsubscr...@hbase.apache.org

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

Reply via email to