Github user srishtyagrawal commented on the issue:
https://github.com/apache/storm/pull/2109
@erikdw Thanks for pointing out the defaults in google_checks.xml. I am not
sure what the reason is behind deviating from the checkstyle defaults.
@revans2 variable `taskID` has been used in the following three files:
1. **KafkaUtils.java:** This file already has a function named `taskId`. I
can rename `taskID` to be `taskid` or `task-id`. Although the variables in this
file follow [Camel case](https://en.wikipedia.org/wiki/Camel_case) naming
standards.
2. **StaticCoordinator.java:** `taskID` can be changed to `taskId` in this
file.
3. **ZkCoordinator.java:** uses `taskId` function, can change the variable
name to `taskid` or `task-id`.
An alternative would be to increase the number of violations.
---
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.
---
