Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1608
@manuzhang Thanks, I addressed your latest comments from
https://github.com/apache/storm/pull/1608/commits/a9f4886fd2f093c7a4450b55ef809fc9826c820e
Not sure we have better representation than '!' without conflicting bash
special char and also not used char for maven artifact.
@satishd Thanks, I addressed your last comment from
https://github.com/apache/storm/pull/1608/commits/17cb2dd333205578bed8f8af8a51072b18daf32d
For custom ACL for jars blobs I'd rather see possible issues first when we
set ACL to blobs. If it's no issue at all and we agree that there could be a
case of this I'll file an issue.
Please take a look again. 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.
---
