You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
374
master passed
+1
compile
56
master passed
+1
checkstyle
77
master passed
+1
shadedjars
264
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
38
master passed
0
spotbugs
242
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
240
master passed
_ Patch Compile Tests _
+1
mvninstall
308
the patch passed
+1
compile
60
the patch passed
+1
javac
60
the patch passed
+1
checkstyle
73
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
307
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1069
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
356
master passed
+1
compile
62
master passed
+1
checkstyle
98
master passed
+1
shadedjars
323
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
42
master passed
0
spotbugs
280
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
278
master passed
_ Patch Compile Tests _
+1
mvninstall
341
the patch passed
+1
compile
60
the patch passed
+1
javac
60
the patch passed
+1
checkstyle
87
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
304
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1039
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
356
master passed
+1
compile
56
master passed
+1
checkstyle
81
master passed
+1
shadedjars
315
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
38
master passed
0
spotbugs
277
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
274
master passed
_ Patch Compile Tests _
+1
mvninstall
360
the patch passed
+1
compile
59
the patch passed
+1
javac
59
the patch passed
+1
checkstyle
77
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
269
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1058
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
389
master passed
+1
compile
77
master passed
+1
checkstyle
111
master passed
+1
shadedjars
359
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
42
master passed
0
spotbugs
308
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
305
master passed
_ Patch Compile Tests _
+1
mvninstall
363
the patch passed
+1
compile
64
the patch passed
+1
javac
64
the patch passed
+1
checkstyle
111
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
346
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1110
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
354
master passed
+1
compile
109
master passed
+1
checkstyle
140
master passed
+1
shadedjars
449
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
101
master passed
0
spotbugs
504
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
500
master passed
_ Patch Compile Tests _
+1
mvninstall
487
the patch passed
+1
compile
137
the patch passed
+1
javac
137
the patch passed
+1
checkstyle
153
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
437
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
987
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
371
master passed
+1
compile
61
master passed
+1
checkstyle
92
master passed
+1
shadedjars
313
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
43
master passed
0
spotbugs
280
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
278
master passed
_ Patch Compile Tests _
+1
mvninstall
342
the patch passed
+1
compile
59
the patch passed
+1
javac
59
the patch passed
+1
checkstyle
90
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
372
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1455
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
363
master passed
+1
compile
61
master passed
+1
checkstyle
93
master passed
+1
shadedjars
307
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
40
master passed
0
spotbugs
320
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
317
master passed
_ Patch Compile Tests _
+1
mvninstall
340
the patch passed
+1
compile
62
the patch passed
+1
javac
62
the patch passed
+1
checkstyle
89
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
308
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1053
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
421
master passed
+1
compile
67
master passed
+1
checkstyle
93
master passed
+1
shadedjars
317
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
43
master passed
0
spotbugs
278
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
276
master passed
_ Patch Compile Tests _
+1
mvninstall
356
the patch passed
+1
compile
61
the patch passed
+1
javac
61
the patch passed
+1
checkstyle
89
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
302
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1051
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
I'm going to close this first approach since it lacks what we need per the continued discussion on HBASE-22874 . I'm only doing this to avoid having the QA bot continue rebuilding the PR every time a commit lands on the target branch. Please continue discussion about needs on HBASE-22874 and let's start with a PR just for master to start dealing with implementation details and look at backports once we get that merged.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1
mvninstall
480
master passed
+1
compile
75
master passed
+1
checkstyle
105
master passed
+1
shadedjars
396
branch has no errors when building our shaded downstream artifacts.
+1
javadoc
53
master passed
0
spotbugs
334
Used deprecated FindBugs config; considering switching to SpotBugs.
+1
findbugs
332
master passed
_ Patch Compile Tests _
+1
mvninstall
474
the patch passed
+1
compile
79
the patch passed
+1
javac
79
the patch passed
+1
checkstyle
97
the patch passed
+1
whitespace
0
The patch has no whitespace issues.
+1
shadedjars
371
patch has no errors when building our shaded downstream artifacts.
+1
hadoopcheck
1298
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@busbey please review.