Skip to content

Fix Checkstyle error, rename a argument #297

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

maobaolong
Copy link
Member

No description provided.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 84 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1189 trunk passed
+1 compile 1334 trunk passed
+1 checkstyle 47 trunk passed
+1 mvnsite 91 trunk passed
+1 shadedclient 834 branch has no errors when building and testing our client artifacts.
+1 javadoc 61 trunk passed
0 spotbugs 140 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 140 trunk passed
_ Patch Compile Tests _
+1 mvninstall 56 the patch passed
+1 compile 1183 the patch passed
+1 javac 1183 the patch passed
+1 checkstyle 38 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 69 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 639 patch has no errors when building and testing our client artifacts.
+1 javadoc 55 the patch passed
+1 findbugs 134 the patch passed
_ Other Tests _
+1 unit 522 hadoop-common in the patch passed.
+1 asflicense 43 The patch does not generate ASF License warnings.
6570
Subsystem Report/Notes
Docker Client=18.09.8 Server=18.09.8 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/1/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 6101423ec763 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 tool maven
Personality personality/hadoop.sh
git revision trunk / 7f1b76c
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/1/testReport/
Max. process+thread count 1386 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/1/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 42 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1094 trunk passed
+1 compile 1075 trunk passed
+1 checkstyle 43 trunk passed
+1 mvnsite 74 trunk passed
+1 shadedclient 746 branch has no errors when building and testing our client artifacts.
+1 javadoc 58 trunk passed
0 spotbugs 116 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 115 trunk passed
_ Patch Compile Tests _
+1 mvninstall 44 the patch passed
+1 compile 958 the patch passed
+1 javac 958 the patch passed
+1 checkstyle 37 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 70 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 592 patch has no errors when building and testing our client artifacts.
+1 javadoc 55 the patch passed
+1 findbugs 123 the patch passed
_ Other Tests _
-1 unit 495 hadoop-common in the patch failed.
+1 asflicense 41 The patch does not generate ASF License warnings.
5704
Reason Tests
Failed junit tests hadoop.ha.TestZKFailoverController
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/2/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 3028e96c4b2d 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 tool maven
Personality personality/hadoop.sh
git revision trunk / ecc8acf
Default Java 1.8.0_212
unit https://builds.apache.org/job/hadoop-multibranch/job/PR-297/2/artifact/out/patch-unit-hadoop-common-project_hadoop-common.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/2/testReport/
Max. process+thread count 1417 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/2/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 63 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1033 trunk passed
+1 compile 1090 trunk passed
+1 checkstyle 38 trunk passed
+1 mvnsite 78 trunk passed
+1 shadedclient 740 branch has no errors when building and testing our client artifacts.
+1 javadoc 55 trunk passed
0 spotbugs 119 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 117 trunk passed
_ Patch Compile Tests _
+1 mvninstall 48 the patch passed
+1 compile 1080 the patch passed
+1 javac 1080 the patch passed
+1 checkstyle 49 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 69 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 608 patch has no errors when building and testing our client artifacts.
+1 javadoc 63 the patch passed
+1 findbugs 119 the patch passed
_ Other Tests _
+1 unit 534 hadoop-common in the patch passed.
+1 asflicense 42 The patch does not generate ASF License warnings.
5891
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/3/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 2b254750326a 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 tool maven
Personality personality/hadoop.sh
git revision trunk / e872ceb
Default Java 1.8.0_222
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/3/testReport/
Max. process+thread count 1390 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/3/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 283 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1404 trunk passed
+1 compile 1292 trunk passed
+1 checkstyle 48 trunk passed
+1 mvnsite 96 trunk passed
+1 shadedclient 1019 branch has no errors when building and testing our client artifacts.
+1 javadoc 77 trunk passed
0 spotbugs 153 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 150 trunk passed
_ Patch Compile Tests _
+1 mvninstall 59 the patch passed
+1 compile 1250 the patch passed
+1 javac 1250 the patch passed
+1 checkstyle 53 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 96 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 762 patch has no errors when building and testing our client artifacts.
+1 javadoc 61 the patch passed
+1 findbugs 131 the patch passed
_ Other Tests _
+1 unit 571 hadoop-common in the patch passed.
+1 asflicense 43 The patch does not generate ASF License warnings.
7471
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/4/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 2c4c151260e9 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 08:28:49 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 00b5a27
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/4/testReport/
Max. process+thread count 1347 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/4/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 38 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1094 trunk passed
+1 compile 1193 trunk passed
+1 checkstyle 37 trunk passed
+1 mvnsite 70 trunk passed
+1 shadedclient 775 branch has no errors when building and testing our client artifacts.
+1 javadoc 59 trunk passed
0 spotbugs 126 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 124 trunk passed
_ Patch Compile Tests _
+1 mvninstall 48 the patch passed
+1 compile 1090 the patch passed
+1 javac 1090 the patch passed
+1 checkstyle 42 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 73 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 660 patch has no errors when building and testing our client artifacts.
+1 javadoc 60 the patch passed
+1 findbugs 126 the patch passed
_ Other Tests _
+1 unit 536 hadoop-common in the patch passed.
+1 asflicense 38 The patch does not generate ASF License warnings.
6123
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/5/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux c70304656830 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 tool maven
Personality personality/hadoop.sh
git revision trunk / e356e4f
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/5/testReport/
Max. process+thread count 1361 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/5/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 196 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1407 trunk passed
+1 compile 1253 trunk passed
+1 checkstyle 56 trunk passed
+1 mvnsite 95 trunk passed
+1 shadedclient 1032 branch has no errors when building and testing our client artifacts.
+1 javadoc 77 trunk passed
0 spotbugs 152 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 149 trunk passed
_ Patch Compile Tests _
+1 mvninstall 60 the patch passed
+1 compile 1131 the patch passed
+1 javac 1131 the patch passed
+1 checkstyle 41 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 74 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 757 patch has no errors when building and testing our client artifacts.
+1 javadoc 70 the patch passed
+1 findbugs 138 the patch passed
_ Other Tests _
+1 unit 555 hadoop-common in the patch passed.
+1 asflicense 43 The patch does not generate ASF License warnings.
7207
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/6/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 3f0d143e9167 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 08:28:49 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 4cb22cd
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/6/testReport/
Max. process+thread count 1346 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/6/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 75 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1444 trunk passed
+1 compile 1442 trunk passed
+1 checkstyle 57 trunk passed
+1 mvnsite 109 trunk passed
+1 shadedclient 1032 branch has no errors when building and testing our client artifacts.
+1 javadoc 75 trunk passed
0 spotbugs 177 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 174 trunk passed
_ Patch Compile Tests _
+1 mvninstall 62 the patch passed
+1 compile 1351 the patch passed
+1 javac 1351 the patch passed
+1 checkstyle 55 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 97 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 893 patch has no errors when building and testing our client artifacts.
+1 javadoc 89 the patch passed
+1 findbugs 170 the patch passed
_ Other Tests _
+1 unit 579 hadoop-common in the patch passed.
+1 asflicense 50 The patch does not generate ASF License warnings.
7832
Subsystem Report/Notes
Docker Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/7/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 6a9f82dcdba6 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 63c295e
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/7/testReport/
Max. process+thread count 1774 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/7/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 83 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1254 trunk passed
+1 compile 1109 trunk passed
+1 checkstyle 46 trunk passed
+1 mvnsite 87 trunk passed
+1 shadedclient 1051 branch has no errors when building and testing our client artifacts.
+1 javadoc 81 trunk passed
0 spotbugs 155 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 152 trunk passed
_ Patch Compile Tests _
+1 mvninstall 69 the patch passed
+1 compile 1342 the patch passed
+1 javac 1342 the patch passed
+1 checkstyle 60 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 98 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 864 patch has no errors when building and testing our client artifacts.
+1 javadoc 87 the patch passed
+1 findbugs 166 the patch passed
_ Other Tests _
+1 unit 653 hadoop-common in the patch passed.
+1 asflicense 62 The patch does not generate ASF License warnings.
7367
Subsystem Report/Notes
Docker Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/8/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 0038aa181c84 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 3329257
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/8/testReport/
Max. process+thread count 1726 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/8/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 44 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1140 trunk passed
+1 compile 1048 trunk passed
+1 checkstyle 42 trunk passed
+1 mvnsite 87 trunk passed
+1 shadedclient 843 branch has no errors when building and testing our client artifacts.
+1 javadoc 71 trunk passed
0 spotbugs 125 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 123 trunk passed
_ Patch Compile Tests _
+1 mvninstall 50 the patch passed
+1 compile 987 the patch passed
+1 javac 987 the patch passed
+1 checkstyle 46 hadoop-common-project/hadoop-common: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 mvnsite 79 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 713 patch has no errors when building and testing our client artifacts.
+1 javadoc 74 the patch passed
+1 findbugs 137 the patch passed
_ Other Tests _
+1 unit 550 hadoop-common in the patch passed.
+1 asflicense 55 The patch does not generate ASF License warnings.
6162
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-297/9/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux a23b58d300f2 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 tool maven
Personality personality/hadoop.sh
git revision trunk / 915cbc9
Default Java 1.8.0_222
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-297/9/testReport/
Max. process+thread count 1400 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-297/9/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
…batch processing

For each run of a batch application, we need to clear the internal streams from the previous run and recreate new ones. This patch introduces the following:
1) bounded flag in StreamSpec
2) app.mode (BATCH/STREAM) in the application config. An application is a batch app iff all the input streams are bounded.
3) app.runId and use it to generate the internal topics for each run.

run.id generation is not addressed in this pr. There will be another patch to resolve it for both yarn and standalone. For now, this patch only works for yarn.

Author: Xinyu Liu <xiliu@xiliu-ld1.linkedin.biz>

Reviewers: Jake Maes <jmakes@apache.org>

Closes apache#297 from xinyuiscool/SAMZA-1417
@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 48s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 1s codespell was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s 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.
_ trunk Compile Tests _
+1 💚 mvninstall 33m 41s trunk passed
+1 💚 compile 23m 32s trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 compile 20m 29s trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+1 💚 checkstyle 1m 5s trunk passed
+1 💚 mvnsite 1m 42s trunk passed
+1 💚 javadoc 1m 8s trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 javadoc 1m 43s trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+1 💚 spotbugs 2m 32s trunk passed
+1 💚 shadedclient 23m 6s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚 mvninstall 1m 2s the patch passed
+1 💚 compile 22m 46s the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 javac 22m 46s the patch passed
+1 💚 compile 20m 37s the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+1 💚 javac 20m 37s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 1m 4s hadoop-common-project/hadoop-common: The patch generated 0 new + 9 unchanged - 1 fixed = 9 total (was 10)
+1 💚 mvnsite 1m 36s the patch passed
+1 💚 javadoc 1m 8s the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 javadoc 1m 38s the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+1 💚 spotbugs 2m 40s the patch passed
+1 💚 shadedclient 23m 22s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 17m 30s hadoop-common in the patch passed.
+1 💚 asflicense 0m 55s The patch does not generate ASF License warnings.
204m 5s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-297/1/artifact/out/Dockerfile
GITHUB PR #297
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell
uname Linux c34b196b1ee1 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 0154398
Default Java Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
Multi-JDK versions /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-297/1/testReport/
Max. process+thread count 3158 (vs. ulimit of 5500)
modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-297/1/console
versions git=2.25.1 maven=3.6.3 spotbugs=4.2.2
Powered by Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants