Github user abhishekagarwal87 commented on the pull request:
https://github.com/apache/storm/pull/1161#issuecomment-190074820
I suspected that the logs directory is being created inside the project
itself and failing rat check. I pulled in your changes to check the same.
```find storm-core/storm-local/logs -type f | wc -l
26
```
This directory is not excluded from the RAT check and thus build is
failing. can you try excluding this directory from RAT check and run the build
on your local end (mvn clean install) to see if the build succeeds?
Sorry to tell this late but I see a problem with these changes I didn't
think of before. Earlier the default log location was under logs directory.
With these changes, logs will start going in storm-local directory which is
meant for internal use. logs directory becomes redundant since storm.loca.dir
always has a default value. In fact this PR will break STORM-1552 which
@arunmahadevan should confirm.
The exceptions in test classes may be fixed without changing the getLogDir
implementation (by passing the `storm.local.dir/logs` as storm.log.dir property
value instead of hardcoded logs directory)
---
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.
---
