[
https://issues.apache.org/jira/browse/HADOOP-16156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16798449#comment-16798449
]
Daryn Sharp commented on HADOOP-16156:
--------------------------------------
Patch looks ok and I'm not objecting to the changes. IMHO, it's not a good
idea to reformat for the mere sake of reformatting. All it will do is cause
someone else a merge headache or even migraine...
That said, if this patch goes in, {{index !=-1}} , is missing a space between
=/-...
> [Clean-up] Remove NULL check before instanceof and fix checkstyle in
> InnerNodeImpl
> ----------------------------------------------------------------------------------
>
> Key: HADOOP-16156
> URL: https://issues.apache.org/jira/browse/HADOOP-16156
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: Shweta
> Assignee: Shweta
> Priority: Minor
> Attachments: HADOOP-16156.001.patch, HADOOP-16156.002.patch,
> HADOOP-16156.003.patch
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
