Skip to content

fix some alerts raised by LGTM - #474

Closed
malcolmtaylor wants to merge 1 commit into
apache:trunkfrom
malcolmtaylor:fix-lgtm-alerts
Closed

fix some alerts raised by LGTM#474
malcolmtaylor wants to merge 1 commit into
apache:trunkfrom
malcolmtaylor:fix-lgtm-alerts

Conversation

@malcolmtaylor

@malcolmtaylormalcolmtaylor commented Jan 30, 2019

Copy link
Copy Markdown
Contributor

LGTM has raised some alerts after analysing Hadoop (https://lgtm.com/projects/g/apache/hadoop/alerts/?mode=tree).
This pull request is to fix some of the more straightforward ones. Specifically,

  • ArrayIndexOutOfBounds in KerberosName
  • Contradictory type checks in GenericExceptionHandler
    -Missing format argument in RegistrySecurity, HttpFSExceptionProvider, DatasetVolumeChecker and ServiceClient

(disclosure: I work for Semmle, the company behind LGTM).

@hadoop-yetus

hadoop-yetus commented Jul 19, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec0Docker mode activated.
-1patch12#474 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.
SubsystemReport/Notes
GITHUB PR#474
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/1/console
versionsgit=2.7.4
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Jul 26, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec42Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep23Maven dependency ordering for branch
+1mvninstall1089trunk passed
+1compile1117trunk passed
+1checkstyle135trunk passed
+1mvnsite144trunk passed
+1shadedclient970branch has no errors when building and testing our client artifacts.
+1javadoc120trunk passed
0spotbugs107Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs231trunk passed
_ Patch Compile Tests _
0mvndep22Maven dependency ordering for patch
+1mvninstall99the patch passed
+1compile1043the patch passed
+1javac1043the patch passed
-0checkstyle130root: The patch generated 1 new + 123 unchanged - 1 fixed = 124 total (was 124)
+1mvnsite146the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient631patch has no errors when building and testing our client artifacts.
+1javadoc119the patch passed
+1findbugs274the patch passed
_ Other Tests _
+1unit187hadoop-auth in the patch passed.
+1unit61hadoop-registry in the patch passed.
+1unit285hadoop-hdfs-httpfs in the patch passed.
+1unit232hadoop-yarn-common in the patch passed.
+1asflicense41The patch does not generate ASF License warnings.
7149
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/2/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux 64e16223de95 4.4.0-157-generic #185-Ubuntu SMP Tue Jul 23 09:17:01 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / c7c7a88
Default Java1.8.0_212
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/2/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/2/testReport/
Max. process+thread count619 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/2/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Aug 2, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec69Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep27Maven dependency ordering for branch
+1mvninstall1269trunk passed
+1compile1271trunk passed
+1checkstyle143trunk passed
+1mvnsite159trunk passed
+1shadedclient1022branch has no errors when building and testing our client artifacts.
+1javadoc129trunk passed
0spotbugs134Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs284trunk passed
_ Patch Compile Tests _
0mvndep28Maven dependency ordering for patch
+1mvninstall115the patch passed
+1compile1149the patch passed
+1javac1149the patch passed
-0checkstyle149root: The patch generated 1 new + 123 unchanged - 1 fixed = 124 total (was 124)
+1mvnsite157the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient676patch has no errors when building and testing our client artifacts.
+1javadoc144the patch passed
+1findbugs314the patch passed
_ Other Tests _
+1unit218hadoop-auth in the patch passed.
+1unit81hadoop-registry in the patch passed.
+1unit333hadoop-hdfs-httpfs in the patch passed.
+1unit246hadoop-yarn-common in the patch passed.
+1asflicense41The patch does not generate ASF License warnings.
8038
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/3/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux 9758ef6e3e6c 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / c2d00c8
Default Java1.8.0_212
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/3/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/3/testReport/
Max. process+thread count616 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/3/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Aug 8, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec40Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep23Maven dependency ordering for branch
+1mvninstall1038trunk passed
+1compile1283trunk passed
+1checkstyle130trunk passed
+1mvnsite139trunk passed
+1shadedclient963branch has no errors when building and testing our client artifacts.
+1javadoc128trunk passed
0spotbugs104Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs224trunk passed
_ Patch Compile Tests _
0mvndep26Maven dependency ordering for patch
+1mvninstall94the patch passed
+1compile1015the patch passed
+1javac1015the patch passed
-0checkstyle131root: The patch generated 1 new + 123 unchanged - 1 fixed = 124 total (was 124)
+1mvnsite139the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient633patch has no errors when building and testing our client artifacts.
+1javadoc122the patch passed
+1findbugs257the patch passed
_ Other Tests _
+1unit189hadoop-auth in the patch passed.
-1unit54hadoop-registry in the patch failed.
-1unit268hadoop-hdfs-httpfs in the patch failed.
+1unit223hadoop-yarn-common in the patch passed.
+1asflicense42The patch does not generate ASF License warnings.
7182
ReasonTests
Failed junit testshadoop.registry.operations.TestRegistryOperations
hadoop.fs.http.client.TestHttpFSWithHttpFSFileSystem
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/4/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux bb13399ad3ba 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / b0131bc
Default Java1.8.0_222
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/4/artifact/out/diff-checkstyle-root.txt
unithttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/4/artifact/out/patch-unit-hadoop-common-project_hadoop-registry.txt
unithttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/4/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs-httpfs.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/4/testReport/
Max. process+thread count626 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/4/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Aug 16, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec83Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep22Maven dependency ordering for branch
+1mvninstall1181trunk passed
+1compile1076trunk passed
+1checkstyle153trunk passed
+1mvnsite153trunk passed
+1shadedclient1100branch has no errors when building and testing our client artifacts.
+1javadoc135trunk passed
0spotbugs116Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs247trunk passed
_ Patch Compile Tests _
0mvndep25Maven dependency ordering for patch
+1mvninstall103the patch passed
+1compile1033the patch passed
+1javac1033the patch passed
-0checkstyle156root: The patch generated 1 new + 123 unchanged - 1 fixed = 124 total (was 124)
+1mvnsite162the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient748patch has no errors when building and testing our client artifacts.
+1javadoc137the patch passed
+1findbugs291the patch passed
_ Other Tests _
+1unit210hadoop-auth in the patch passed.
+1unit63hadoop-registry in the patch passed.
+1unit282hadoop-hdfs-httpfs in the patch passed.
+1unit233hadoop-yarn-common in the patch passed.
+1asflicense54The patch does not generate ASF License warnings.
7662
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/5/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux ddc0ab9672ca 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / e356e4f
Default Java1.8.0_222
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/5/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/5/testReport/
Max. process+thread count607 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/5/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Aug 20, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec46Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep23Maven dependency ordering for branch
+1mvninstall1115trunk passed
+1compile1076trunk passed
+1checkstyle137trunk passed
+1mvnsite141trunk passed
+1shadedclient1016branch has no errors when building and testing our client artifacts.
+1javadoc112trunk passed
0spotbugs98Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs210trunk passed
_ Patch Compile Tests _
0mvndep22Maven dependency ordering for patch
+1mvninstall99the patch passed
+1compile1100the patch passed
+1javac1100the patch passed
-0checkstyle144root: The patch generated 1 new + 124 unchanged - 1 fixed = 125 total (was 125)
+1mvnsite130the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient632patch has no errors when building and testing our client artifacts.
+1javadoc132the patch passed
+1findbugs257the patch passed
_ Other Tests _
+1unit183hadoop-auth in the patch passed.
+1unit57hadoop-registry in the patch passed.
+1unit260hadoop-hdfs-httpfs in the patch passed.
+1unit218hadoop-yarn-common in the patch passed.
+1asflicense45The patch does not generate ASF License warnings.
7166
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/6/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux dc84cf67c6be 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / 094d736
Default Java1.8.0_212
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/6/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/6/testReport/
Max. process+thread count624 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/6/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Aug 22, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec61Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep20Maven dependency ordering for branch
+1mvninstall1058trunk passed
+1compile1071trunk passed
+1checkstyle138trunk passed
+1mvnsite157trunk passed
+1shadedclient1001branch has no errors when building and testing our client artifacts.
+1javadoc113trunk passed
0spotbugs98Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs207trunk passed
_ Patch Compile Tests _
0mvndep22Maven dependency ordering for patch
+1mvninstall98the patch passed
+1compile1145the patch passed
+1javac1145the patch passed
-0checkstyle141root: The patch generated 1 new + 123 unchanged - 1 fixed = 124 total (was 124)
+1mvnsite139the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient618patch has no errors when building and testing our client artifacts.
+1javadoc122the patch passed
+1findbugs257the patch passed
_ Other Tests _
+1unit203hadoop-auth in the patch passed.
+1unit61hadoop-registry in the patch passed.
-1unit294hadoop-hdfs-httpfs in the patch failed.
+1unit229hadoop-yarn-common in the patch passed.
+1asflicense41The patch does not generate ASF License warnings.
7186
ReasonTests
Failed junit testshadoop.fs.http.server.TestHttpFSServer
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/7/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux 791dd235f301 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / 69ddb36
Default Java1.8.0_222
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/7/artifact/out/diff-checkstyle-root.txt
unithttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/7/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs-httpfs.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/7/testReport/
Max. process+thread count615 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/7/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Aug 27, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec61Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep25Maven dependency ordering for branch
+1mvninstall1171trunk passed
+1compile1095trunk passed
+1checkstyle148trunk passed
+1mvnsite168trunk passed
+1shadedclient1055branch has no errors when building and testing our client artifacts.
+1javadoc154trunk passed
0spotbugs112Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs256trunk passed
_ Patch Compile Tests _
0mvndep22Maven dependency ordering for patch
+1mvninstall114the patch passed
+1compile972the patch passed
+1javac972the patch passed
-0checkstyle142root: The patch generated 1 new + 124 unchanged - 1 fixed = 125 total (was 125)
+1mvnsite174the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient684patch has no errors when building and testing our client artifacts.
+1javadoc162the patch passed
+1findbugs289the patch passed
_ Other Tests _
+1unit199hadoop-auth in the patch passed.
+1unit71hadoop-registry in the patch passed.
+1unit307hadoop-hdfs-httpfs in the patch passed.
+1unit239hadoop-yarn-common in the patch passed.
+1asflicense50The patch does not generate ASF License warnings.
7590
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/8/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux 5f2df5267072 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / 3329257
Default Java1.8.0_222
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/8/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/8/testReport/
Max. process+thread count624 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/8/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

hadoop-yetus commented Sep 3, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec35Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1@author0The patch does not contain any @author tags.
-1test4tests0The 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.
_ trunk Compile Tests _
0mvndep22Maven dependency ordering for branch
+1mvninstall1167trunk passed
+1compile1064trunk passed
+1checkstyle152trunk passed
+1mvnsite160trunk passed
+1shadedclient1103branch has no errors when building and testing our client artifacts.
+1javadoc137trunk passed
0spotbugs121Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs250trunk passed
_ Patch Compile Tests _
0mvndep24Maven dependency ordering for patch
+1mvninstall107the patch passed
+1compile1030the patch passed
+1javac1030the patch passed
-0checkstyle158root: The patch generated 1 new + 123 unchanged - 1 fixed = 124 total (was 124)
+1mvnsite151the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedclient748patch has no errors when building and testing our client artifacts.
+1javadoc140the patch passed
+1findbugs307the patch passed
_ Other Tests _
+1unit200hadoop-auth in the patch passed.
+1unit62hadoop-registry in the patch passed.
+1unit336hadoop-hdfs-httpfs in the patch passed.
+1unit238hadoop-yarn-common in the patch passed.
+1asflicense47The patch does not generate ASF License warnings.
7643
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-474/9/artifact/out/Dockerfile
GITHUB PR#474
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
unameLinux 92ba678b92a3 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / 915cbc9
Default Java1.8.0_222
checkstylehttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/9/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/9/testReport/
Max. process+thread count599 (vs. ulimit of 5500)
modulesC: hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-474/9/console
versionsgit=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@aajisaka

aajisaka commented Sep 11, 2019

Copy link
Copy Markdown
Member

Hi @1m2c3t4
Would you create an issue in ASF JIRA before creating a pull request?
Contributor guide: https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute

@malcolmtaylor

malcolmtaylor commented Sep 12, 2019

Copy link
Copy Markdown
ContributorAuthor

@aajisaka I have now created HADOOP-16556. I'll close this PR and replace it with #1428

shanthoosh added a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
@shanthoosh@jagadish-v0
SAMZA-1656: EventHubSystemAdmin does not fetch metadata for valid str…
71dba52
…eams.
Currently successive invocation of EventHubSystemAdmin.getSystemStreamMetadata with the same stream collection returns empty results.
This is an implementation bug, where the streams requested as a part of prior invocations of EventHubSystemAdmin.getSystemStreamMetadata are ignored(due to stale caching).
This bug causes the JobModel generation phase to fail and kills the StreamProcessor.
Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com>
Reviewers: Jagadish <jagadish@apache.org>, Srini P<spunuru@linkedin.com>
Closesapache#474 from shanthoosh/EventHubSystemAdminBug
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@malcolmtaylor@hadoop-yetus@aajisaka