Github user apiri commented on the pull request:
https://github.com/apache/nifi/pull/91#issuecomment-167644662
@rickysaltzer Sounds fair. I had squashed/pushed to my personal fork at
a35be5e358108a48b34de3fcf5edcb428a794d03 to make the review a little easier.
Additionally, there were a couple of style items I patched up as well as
marking OnScheduled/OnStopped variables as volatile in
1d4fdfe57c9d966c0be6fd5cc537623854f4491c. If they seem fair, I can merge them
to accompany your contribution and push it all together.
Let me know what works and sounds good to you. Thanks!
---
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.
---
