You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "guluo2016 (via GitHub)" <gi...@apache.org> on 2026/03/07 05:58:22 UTC
[PR] HBASE-29978 Remove redundant imports in regionserver.jsp file [hbase]
guluo2016 opened a new pull request, #7880:
URL: https://github.com/apache/hbase/pull/7880
Details see: [HBASE-29978](https://issues.apache.org/jira/browse/HBASE-29978)
--
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-unsubscribe@hbase.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] HBASE-29978 Remove redundant imports in regionserver.jsp file [hbase]
Posted by "guluo2016 (via GitHub)" <gi...@apache.org>.
guluo2016 commented on code in PR #7880:
URL: https://github.com/apache/hbase/pull/7880#discussion_r2910816919
##########
hbase-server/src/main/resources/hbase-webapps/regionserver/regionserver.jsp:
##########
@@ -19,7 +19,6 @@
--%>
<%@ page contentType="text/html;charset=UTF-8"
import="java.util.*"
Review Comment:
I noticed this as well.
Actually, similar imports also exist in several other JSP files.
Maybe we can open a separate Jira issue to address this globally.
--
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-unsubscribe@hbase.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] HBASE-29978 Remove redundant imports in regionserver.jsp file [hbase]
Posted by "NihalJain (via GitHub)" <gi...@apache.org>.
NihalJain commented on code in PR #7880:
URL: https://github.com/apache/hbase/pull/7880#discussion_r2910996405
##########
hbase-server/src/main/resources/hbase-webapps/regionserver/regionserver.jsp:
##########
@@ -19,7 +19,6 @@
--%>
<%@ page contentType="text/html;charset=UTF-8"
import="java.util.*"
Review Comment:
sounds good
--
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-unsubscribe@hbase.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] HBASE-29978 Remove redundant imports in regionserver.jsp file [hbase]
Posted by "guluo2016 (via GitHub)" <gi...@apache.org>.
guluo2016 merged PR #7880:
URL: https://github.com/apache/hbase/pull/7880
--
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-unsubscribe@hbase.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] HBASE-29978 Remove redundant imports in regionserver.jsp file [hbase]
Posted by "guluo2016 (via GitHub)" <gi...@apache.org>.
guluo2016 commented on PR #7880:
URL: https://github.com/apache/hbase/pull/7880#issuecomment-4031956330
Thanks @NihalJain @liuxiaocs7 @vaijosh for the review.
Merging it now!
--
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-unsubscribe@hbase.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] HBASE-29978 Remove redundant imports in regionserver.jsp file [hbase]
Posted by "NihalJain (via GitHub)" <gi...@apache.org>.
NihalJain commented on code in PR #7880:
URL: https://github.com/apache/hbase/pull/7880#discussion_r2909553247
##########
hbase-server/src/main/resources/hbase-webapps/regionserver/regionserver.jsp:
##########
@@ -19,7 +19,6 @@
--%>
<%@ page contentType="text/html;charset=UTF-8"
import="java.util.*"
Review Comment:
nit: not related. maybe get rid of .* imports
--
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-unsubscribe@hbase.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org