Skip to content
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

Fix checkstyle problem #295

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

Conversation

maobaolong
Copy link
Member

No description provided.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 35 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 1037 trunk passed
+1 compile 1039 trunk passed
+1 checkstyle 37 trunk passed
+1 mvnsite 73 trunk passed
+1 shadedclient 775 branch has no errors when building and testing our client artifacts.
+1 javadoc 58 trunk passed
0 spotbugs 118 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 117 trunk passed
_ Patch Compile Tests _
+1 mvninstall 49 the patch passed
+1 compile 996 the patch passed
+1 javac 996 the patch passed
+1 checkstyle 39 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 69 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 613 patch has no errors when building and testing our client artifacts.
+1 javadoc 55 the patch passed
+1 findbugs 126 the patch passed
_ Other Tests _
-1 unit 506 hadoop-common in the patch failed.
+1 asflicense 40 The patch does not generate ASF License warnings.
5720
Reason Tests
Failed junit tests hadoop.util.TestBasicDiskValidator
Subsystem Report/Notes
Docker Client=18.09.8 Server=18.09.8 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/1/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux d2292252882e 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
unit https://builds.apache.org/job/hadoop-multibranch/job/PR-295/1/artifact/out/patch-unit-hadoop-common-project_hadoop-common.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/1/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-295/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 41 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 1059 trunk passed
+1 compile 1028 trunk passed
+1 checkstyle 37 trunk passed
+1 mvnsite 73 trunk passed
+1 shadedclient 810 branch has no errors when building and testing our client artifacts.
+1 javadoc 64 trunk passed
0 spotbugs 120 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 117 trunk passed
_ Patch Compile Tests _
+1 mvninstall 45 the patch passed
+1 compile 972 the patch passed
+1 javac 972 the patch passed
+1 checkstyle 36 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 70 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 657 patch has no errors when building and testing our client artifacts.
+1 javadoc 60 the patch passed
+1 findbugs 125 the patch passed
_ Other Tests _
+1 unit 511 hadoop-common in the patch passed.
+1 asflicense 41 The patch does not generate ASF License warnings.
5797
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/2/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 0a84f7b8028e 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 / 62efb63
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/2/testReport/
Max. process+thread count 1382 (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-295/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 40 Docker mode activated.
_ Prechecks _
+1 dupname 1 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 1061 trunk passed
+1 compile 1067 trunk passed
+1 checkstyle 35 trunk passed
+1 mvnsite 71 trunk passed
+1 shadedclient 732 branch has no errors when building and testing our client artifacts.
+1 javadoc 61 trunk passed
0 spotbugs 120 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 118 trunk passed
_ Patch Compile Tests _
+1 mvninstall 45 the patch passed
+1 compile 1061 the patch passed
+1 javac 1061 the patch passed
+1 checkstyle 38 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 70 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 614 patch has no errors when building and testing our client artifacts.
+1 javadoc 58 the patch passed
+1 findbugs 123 the patch passed
_ Other Tests _
+1 unit 532 hadoop-common in the patch passed.
+1 asflicense 42 The patch does not generate ASF License warnings.
5838
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/3/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux da06a6412b8b 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 tool maven
Personality personality/hadoop.sh
git revision trunk / e872ceb
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/3/testReport/
Max. process+thread count 1430 (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-295/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 82 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 1256 trunk passed
+1 compile 1070 trunk passed
+1 checkstyle 42 trunk passed
+1 mvnsite 81 trunk passed
+1 shadedclient 899 branch has no errors when building and testing our client artifacts.
+1 javadoc 64 trunk passed
0 spotbugs 125 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 123 trunk passed
_ Patch Compile Tests _
+1 mvninstall 52 the patch passed
+1 compile 1068 the patch passed
+1 javac 1068 the patch passed
+1 checkstyle 46 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 85 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 759 patch has no errors when building and testing our client artifacts.
+1 javadoc 63 the patch passed
+1 findbugs 135 the patch passed
_ Other Tests _
+1 unit 588 hadoop-common in the patch passed.
+1 asflicense 43 The patch does not generate ASF License warnings.
6522
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/4/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux a02f15c315ff 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 tool maven
Personality personality/hadoop.sh
git revision trunk / 00b5a27
Default Java 1.8.0_222
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/4/testReport/
Max. process+thread count 1356 (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-295/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 78 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 1258 trunk passed
+1 compile 1128 trunk passed
+1 checkstyle 39 trunk passed
+1 mvnsite 79 trunk passed
+1 shadedclient 852 branch has no errors when building and testing our client artifacts.
+1 javadoc 60 trunk passed
0 spotbugs 118 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 117 trunk passed
_ Patch Compile Tests _
+1 mvninstall 48 the patch passed
+1 compile 1113 the patch passed
+1 javac 1113 the patch passed
+1 checkstyle 39 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 72 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 730 patch has no errors when building and testing our client artifacts.
+1 javadoc 60 the patch passed
+1 findbugs 128 the patch passed
_ Other Tests _
+1 unit 588 hadoop-common in the patch passed.
+1 asflicense 51 The patch does not generate ASF License warnings.
6503
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/5/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 2fcebe7c218c 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 / e356e4f
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/5/testReport/
Max. process+thread count 1349 (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-295/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 87 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 1250 trunk passed
+1 compile 1445 trunk passed
+1 checkstyle 54 trunk passed
+1 mvnsite 107 trunk passed
+1 shadedclient 1038 branch has no errors when building and testing our client artifacts.
+1 javadoc 76 trunk passed
0 spotbugs 149 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 146 trunk passed
_ Patch Compile Tests _
+1 mvninstall 56 the patch passed
+1 compile 1294 the patch passed
+1 javac 1294 the patch passed
+1 checkstyle 57 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 106 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 870 patch has no errors when building and testing our client artifacts.
+1 javadoc 79 the patch passed
+1 findbugs 153 the patch passed
_ Other Tests _
+1 unit 653 hadoop-common in the patch passed.
+1 asflicense 45 The patch does not generate ASF License warnings.
7577
Subsystem Report/Notes
Docker Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/6/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 80b93c896dfd 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 / 4cb22cd
Default Java 1.8.0_212
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/6/testReport/
Max. process+thread count 1776 (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-295/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 50 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 1241 trunk passed
+1 compile 1324 trunk passed
+1 checkstyle 44 trunk passed
+1 mvnsite 84 trunk passed
+1 shadedclient 920 branch has no errors when building and testing our client artifacts.
+1 javadoc 68 trunk passed
0 spotbugs 146 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 143 trunk passed
_ Patch Compile Tests _
+1 mvninstall 58 the patch passed
+1 compile 1222 the patch passed
+1 javac 1222 the patch passed
+1 checkstyle 44 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 79 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 677 patch has no errors when building and testing our client artifacts.
+1 javadoc 64 the patch passed
+1 findbugs 136 the patch passed
_ Other Tests _
+1 unit 594 hadoop-common in the patch passed.
+1 asflicense 45 The patch does not generate ASF License warnings.
6848
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/7/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux bf98a3beee16 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 / 63c295e
Default Java 1.8.0_222
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/7/testReport/
Max. process+thread count 1447 (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-295/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 39 Docker mode activated.
_ Prechecks _
+1 dupname 1 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 1016 trunk passed
+1 compile 963 trunk passed
+1 checkstyle 49 trunk passed
+1 mvnsite 81 trunk passed
+1 shadedclient 810 branch has no errors when building and testing our client artifacts.
+1 javadoc 73 trunk passed
0 spotbugs 119 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 118 trunk passed
_ Patch Compile Tests _
+1 mvninstall 46 the patch passed
+1 compile 925 the patch passed
+1 javac 925 the patch passed
+1 checkstyle 49 hadoop-common-project/hadoop-common: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)
+1 mvnsite 80 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 683 patch has no errors when building and testing our client artifacts.
+1 javadoc 69 the patch passed
+1 findbugs 125 the patch passed
_ Other Tests _
+1 unit 564 hadoop-common in the patch passed.
+1 asflicense 52 The patch does not generate ASF License warnings.
5822
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-295/8/artifact/out/Dockerfile
GITHUB PR #295
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux 5a2068699c09 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 / 3329257
Default Java 1.8.0_222
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-295/8/testReport/
Max. process+thread count 1378 (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-295/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.

shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
## Samza SQL implementation for basic projects, filtering and

## Design document:
https://docs.google.com/document/d/1bE-ZuPfTpntm1hT3GwQEShYDiTqU3IkxeP4-3ZcGHgU/edit?usp=sharing

Author: Srinivasulu Punuru <spunuru@linkedin.com>

Reviewers: Yi Pan <nickpan47@gmail.com>

Closes apache#295 from srinipunuru/samza-sql.1
NyteKnight pushed a commit to NyteKnight/hadoop that referenced this pull request Jun 25, 2024
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