You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Akira Ajisaka <aa...@apache.org> on 2016/03/30 04:02:54 UTC
Review Request 45374: Use ">>" instead of ">" to write
ambari-metrics-collector.out
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45374/
-----------------------------------------------------------
Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle.
Bugs: AMBARI-15588
https://issues.apache.org/jira/browse/AMBARI-15588
Repository: ambari
Description
-------
If we want to rotate ambari-metrics-collector.out by logrotate (using copytruncate option), we need to use ">>" instead of ">" not to keep the file offset to write. If the offset is kept, null characters are padded in the new file.
Diffs
-----
ambari-metrics/ambari-metrics-host-monitoring/conf/unix/ambari-metrics-monitor 7464c55
Diff: https://reviews.apache.org/r/45374/diff/
Testing
-------
Manually tested the patch and verified that the .out is rotated collectly.
Thanks,
Akira Ajisaka
Re: Review Request 45374: Use ">>" instead of ">" to write
ambari-metrics-collector.out
Posted by Dmytro Sen <ds...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45374/#review126084
-----------------------------------------------------------
Ship it!
I'm ok with that, but lets wait for one more +1
- Dmytro Sen
On Март 30, 2016, 2:02 д.п., Akira Ajisaka wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45374/
> -----------------------------------------------------------
>
> (Updated Март 30, 2016, 2:02 д.п.)
>
>
> Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle.
>
>
> Bugs: AMBARI-15588
> https://issues.apache.org/jira/browse/AMBARI-15588
>
>
> Repository: ambari
>
>
> Description
> -------
>
> If we want to rotate ambari-metrics-collector.out by logrotate (using copytruncate option), we need to use ">>" instead of ">" not to keep the file offset to write. If the offset is kept, null characters are padded in the new file.
>
>
> Diffs
> -----
>
> ambari-metrics/ambari-metrics-host-monitoring/conf/unix/ambari-metrics-monitor 7464c55
>
> Diff: https://reviews.apache.org/r/45374/diff/
>
>
> Testing
> -------
>
> Manually tested the patch and verified that the .out is rotated collectly.
>
>
> Thanks,
>
> Akira Ajisaka
>
>
Re: Review Request 45374: Use ">>" instead of ">" to write
ambari-metrics-collector.out
Posted by Aravindan Vijayan <av...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45374/#review126492
-----------------------------------------------------------
Ship it!
Ship It!
- Aravindan Vijayan
On March 30, 2016, 2:02 a.m., Akira Ajisaka wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45374/
> -----------------------------------------------------------
>
> (Updated March 30, 2016, 2:02 a.m.)
>
>
> Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle.
>
>
> Bugs: AMBARI-15588
> https://issues.apache.org/jira/browse/AMBARI-15588
>
>
> Repository: ambari
>
>
> Description
> -------
>
> If we want to rotate ambari-metrics-collector.out by logrotate (using copytruncate option), we need to use ">>" instead of ">" not to keep the file offset to write. If the offset is kept, null characters are padded in the new file.
>
>
> Diffs
> -----
>
> ambari-metrics/ambari-metrics-host-monitoring/conf/unix/ambari-metrics-monitor 7464c55
>
> Diff: https://reviews.apache.org/r/45374/diff/
>
>
> Testing
> -------
>
> Manually tested the patch and verified that the .out is rotated collectly.
>
>
> Thanks,
>
> Akira Ajisaka
>
>
Re: Review Request 45374: Use ">>" instead of ">" to write
ambari-metrics-collector.out
Posted by Dmytro Sen <ds...@hortonworks.com>.
> On Апрель 6, 2016, 9:05 д.п., Akira Ajisaka wrote:
> > Can someone commit it?
Committed to trunk.
Thanks Akira.
- Dmytro
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45374/#review127304
-----------------------------------------------------------
On Март 30, 2016, 2:02 д.п., Akira Ajisaka wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45374/
> -----------------------------------------------------------
>
> (Updated Март 30, 2016, 2:02 д.п.)
>
>
> Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle.
>
>
> Bugs: AMBARI-15588
> https://issues.apache.org/jira/browse/AMBARI-15588
>
>
> Repository: ambari
>
>
> Description
> -------
>
> If we want to rotate ambari-metrics-collector.out by logrotate (using copytruncate option), we need to use ">>" instead of ">" not to keep the file offset to write. If the offset is kept, null characters are padded in the new file.
>
>
> Diffs
> -----
>
> ambari-metrics/ambari-metrics-host-monitoring/conf/unix/ambari-metrics-monitor 7464c55
>
> Diff: https://reviews.apache.org/r/45374/diff/
>
>
> Testing
> -------
>
> Manually tested the patch and verified that the .out is rotated collectly.
>
>
> Thanks,
>
> Akira Ajisaka
>
>
Re: Review Request 45374: Use ">>" instead of ">" to write
ambari-metrics-collector.out
Posted by Akira Ajisaka <aa...@apache.org>.
> On 4月 6, 2016, 9:05 a.m., Akira Ajisaka wrote:
> > Can someone commit it?
>
> Dmytro Sen wrote:
> Committed to trunk.
>
> Thanks Akira.
Thank you, Dmytro and Aravindan!
- Akira
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45374/#review127304
-----------------------------------------------------------
On 3月 30, 2016, 2:02 a.m., Akira Ajisaka wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45374/
> -----------------------------------------------------------
>
> (Updated 3月 30, 2016, 2:02 a.m.)
>
>
> Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle.
>
>
> Bugs: AMBARI-15588
> https://issues.apache.org/jira/browse/AMBARI-15588
>
>
> Repository: ambari
>
>
> Description
> -------
>
> If we want to rotate ambari-metrics-collector.out by logrotate (using copytruncate option), we need to use ">>" instead of ">" not to keep the file offset to write. If the offset is kept, null characters are padded in the new file.
>
>
> Diffs
> -----
>
> ambari-metrics/ambari-metrics-host-monitoring/conf/unix/ambari-metrics-monitor 7464c55
>
> Diff: https://reviews.apache.org/r/45374/diff/
>
>
> Testing
> -------
>
> Manually tested the patch and verified that the .out is rotated collectly.
>
>
> Thanks,
>
> Akira Ajisaka
>
>
Re: Review Request 45374: Use ">>" instead of ">" to write
ambari-metrics-collector.out
Posted by Akira Ajisaka <aa...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45374/#review127304
-----------------------------------------------------------
Can someone commit it?
- Akira Ajisaka
On 3月 30, 2016, 2:02 a.m., Akira Ajisaka wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45374/
> -----------------------------------------------------------
>
> (Updated 3月 30, 2016, 2:02 a.m.)
>
>
> Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle.
>
>
> Bugs: AMBARI-15588
> https://issues.apache.org/jira/browse/AMBARI-15588
>
>
> Repository: ambari
>
>
> Description
> -------
>
> If we want to rotate ambari-metrics-collector.out by logrotate (using copytruncate option), we need to use ">>" instead of ">" not to keep the file offset to write. If the offset is kept, null characters are padded in the new file.
>
>
> Diffs
> -----
>
> ambari-metrics/ambari-metrics-host-monitoring/conf/unix/ambari-metrics-monitor 7464c55
>
> Diff: https://reviews.apache.org/r/45374/diff/
>
>
> Testing
> -------
>
> Manually tested the patch and verified that the .out is rotated collectly.
>
>
> Thanks,
>
> Akira Ajisaka
>
>