Github user chandnisingh commented on the pull request:
https://github.com/apache/incubator-apex-core/pull/309#issuecomment-210743348
@tweise In this PR we reverted the change that would have required the
release of a new artifact. So if we include this change then we don't have to
release the new checkstyle artifact.
Right now there is a problem with Checkstyle for which we have to file a
bug with them. However this bug affected only 2 usages - PhysicalPlan and DTCli
because of which while fixing the violations in engine,
```AnnotationLocation``` check was suppressed in all the classes.
Suppressing it for all classes will lead to introduction of valid
violations in new changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
