[
https://issues.apache.org/jira/browse/APEXCORE-422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15243040#comment-15243040
]
ASF GitHub Bot commented on APEXCORE-422:
-----------------------------------------
Github user vrozov commented on the pull request:
https://github.com/apache/incubator-apex-core/pull/309#issuecomment-210487635
@tweise I would suggest to merge it before 3.4.0 goes out. Currently
checkstyle definition xml is out of sync with the
apex-codestyle-config-1.0.0-incubating.jar published to maven. PR fixes
inconsistency by reverting back the change in checkstyle definition. The second
change can wait, but it is not clear what is the cons of not pulling it in.
> Checkstyle rule related to allowSamelineParameterizedAnnotation should be
> just suppressed for DTCli and PhysicalPlan
> --------------------------------------------------------------------------------------------------------------------
>
> Key: APEXCORE-422
> URL: https://issues.apache.org/jira/browse/APEXCORE-422
> Project: Apache Apex Core
> Issue Type: Bug
> Reporter: Chandni Singh
> Assignee: Chandni Singh
> Fix For: 3.4.0
>
>
> The ticket was open to address the following issue which was reverted:
> Right now we are not letting users to write the following
> {code}
> for (@SuppressWarnings("deprecation")
> com.datatorrent.api.StatsListener.OperatorCommand cmd : rsp
> .operatorCommands)
> {code}
> We should suppress this rule only for DtCli and Physical Plan
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)