[jira] [Commented] (MAPREDUCE-7188) [Clean-up] Remove NULL check before instanceof in TaskResult
[ https://issues.apache.org/jira/browse/MAPREDUCE-7188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16782041#comment-16782041 ] Hadoop QA commented on MAPREDUCE-7188: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 36s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} 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. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 17m 19s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 23s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 16s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 26s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 25s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 37s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 21s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 24s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 23s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 23s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 12s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 22s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 11m 41s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 16s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 40s{color} | {color:green} hadoop-mapreduce-examples in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 23s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 46m 18s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:8f97d6f | | JIRA Issue | MAPREDUCE-7188 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12960820/MAPREDUCE-7188.001.patch | | Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 6fd81dc4c3d8 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 | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / cab8529 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_191 | | findbugs | v3.1.0-RC1 | | Test Results | https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/7590/testReport/ | | Max. process+thread count | 448 (vs. ulimit of 1) | | modules | C: hadoop-mapreduce-project/hadoop-mapreduce-examples U: hadoop-mapreduce-project/hadoop-mapreduce-examples | | Console output | https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/7590/console | | Powered by | Apache Yetus 0.8.0 http://yetus.apache.org | Thi
[jira] [Commented] (MAPREDUCE-7188) [Clean-up] Remove NULL check before instanceof in TaskResult
[ https://issues.apache.org/jira/browse/MAPREDUCE-7188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16783401#comment-16783401 ] Adam Antal commented on MAPREDUCE-7188: --- Hi [~shwetayakkali], Could you assign this issue to yourself? If the option is not there, you probably need to get access to project MAPREDUCE (possibly from a committer). > [Clean-up] Remove NULL check before instanceof in TaskResult > > > Key: MAPREDUCE-7188 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-7188 > Project: Hadoop Map/Reduce > Issue Type: Improvement >Reporter: Shweta >Priority: Minor > Attachments: MAPREDUCE-7188.001.patch > > -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org
[jira] [Commented] (MAPREDUCE-7188) [Clean-up] Remove NULL check before instanceof in TaskResult
[ https://issues.apache.org/jira/browse/MAPREDUCE-7188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16783413#comment-16783413 ] Adam Antal commented on MAPREDUCE-7188: --- The patch LGTM. Did you think about cleaning up the whole file a bit? Three items I would be happy about being done: * remove default constructor (to me it looks like this is never used, but needs to be proven) * checkstyle: there are a few long (>80 char) lines, that could be broken up (are there other checkstyle issues?) * adding spaces in the ternary operator in {{combine()}} > [Clean-up] Remove NULL check before instanceof in TaskResult > > > Key: MAPREDUCE-7188 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-7188 > Project: Hadoop Map/Reduce > Issue Type: Improvement >Reporter: Shweta >Priority: Minor > Attachments: MAPREDUCE-7188.001.patch > > -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org
[jira] [Commented] (MAPREDUCE-7188) [Clean-up] Remove NULL check before instanceof in TaskResult
[ https://issues.apache.org/jira/browse/MAPREDUCE-7188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16783951#comment-16783951 ] Shweta commented on MAPREDUCE-7188: --- Thank you [~adam.antal] for the review. Yes, the default constructor isn't used anywhere as found by code inspection by IntelliJ. I have fixed the checkstyles issues as you mentioned above. Uploaded patch v002. > [Clean-up] Remove NULL check before instanceof in TaskResult > > > Key: MAPREDUCE-7188 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-7188 > Project: Hadoop Map/Reduce > Issue Type: Improvement >Reporter: Shweta >Priority: Minor > Attachments: MAPREDUCE-7188.001.patch, MAPREDUCE-7188.002.patch > > -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org