[
https://issues.apache.org/jira/browse/TS-3960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15011476#comment-15011476
]
ASF GitHub Bot commented on TS-3960:
------------------------------------
Github user jpeach commented on the pull request:
https://github.com/apache/trafficserver/pull/301#issuecomment-157786700
> 1. How to clean up the old watched children files correctly?
If you have a tree of ```Rollback``` objects, then any object in the tree
that is updated would trigger the root of the tree and remove all the children.
You always know the parent of the watched children, so when you trigger on the
group, you get the parent and remove the children. IIRC you are already doing
this.
> 2. In a parent with all children style, the whole information can be
packed in one message
> instead of O(children) messages, which is more efficient on both time and
space.
Since the message to ```traffic_manager``` is asynchronous and not
performance sensitive, there's no need to try to optimize this. We need to
optimize for simplicity. When you use a single item per message, it is easy to
re-use the existing message marshaling code, and the code in the SSL layer that
collects all the child files is no longer needed.
Although I agree that the tree is more general than strictly necessary, I
think that the code for that will turn out to be fairly clean. I am OK with
file groups, but please add a back pointer from the file group to the parent
```Rollback``` so that the parent lookups can be avoided. Note that if you make
the ```ConfigFileGroup``` a member of ```Rollback``` then you basically have
the tree structure already, which is why I'm suggesting it :)
> traffic_line -x doesn't reload SSL certs content
> ------------------------------------------------
>
> Key: TS-3960
> URL: https://issues.apache.org/jira/browse/TS-3960
> Project: Traffic Server
> Issue Type: Bug
> Components: SSL
> Reporter: Zhang Zizhong
> Assignee: Brian Geffon
> Fix For: 6.1.0
>
>
> traffic_line -x doesn't reload when SSL certs change file contents without
> changing the file names.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
