ndimiduk commented on a change in pull request #4133:
URL: https://github.com/apache/hbase/pull/4133#discussion_r815735085
##########
File path:
hbase-endpoint/src/main/java/org/apache/hadoop/hbase/coprocessor/AggregateImplementation.java
##########
@@ -53,11 +53,11 @@
* {@link ColumnInterpreter} is used to interpret column value. This class is
* parameterized with the following (these are the types with which the {@link
ColumnInterpreter}
* is parameterized, and for more description on these, refer to {@link
ColumnInterpreter}):
- * @param T Cell value data type
- * @param S Promoted data type
- * @param P PB message that is used to transport initializer specific bytes
- * @param Q PB message that is used to transport Cell (<T>) instance
- * @param R PB message that is used to transport Promoted (<S>) instance
+ * @param <T> Cell value data type
Review comment:
No, checking the precommit output, I see no new javadoc or checkstyle
warnings originating from this file.
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/security/HBaseSaslRpcServer.java
##########
@@ -68,7 +68,7 @@ public void dispose() {
public String getAttemptingUser() {
Optional<UserGroupInformation> optionalUser =
serverWithProvider.getAttemptingUser();
if (optionalUser.isPresent()) {
- optionalUser.get().toString();
+ return optionalUser.get().toString();
Review comment:
Yes perhaps. The only use of this method appears to be an "auth failed
for" message sent to the audit log, from `ServerRpcConnection`.
https://issues.apache.org/jira/browse/HBASE-26776
#4138
##########
File path:
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestMergesSplitsAddToTracker.java
##########
@@ -228,7 +228,7 @@ private void validateDaughterRegionsFiles(HRegion region,
String orignalFileName
List<StoreFileInfo> infos =
region.getRegionFileSystem().getStoreFiles("info");
final MutableBoolean foundLink = new MutableBoolean(false);
infos.stream().forEach(i -> {
- i.getActiveFileName().contains(orignalFileName);
+ assertTrue(i.getActiveFileName().contains(orignalFileName));
if(i.getActiveFileName().contains(untrackedFile)){
Review comment:
this change is incorrect. pushing a fix.
--
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]
