<p>Nir Simionovich (GreenfieldTech - Israel) <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6819">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I raised this review on IRC, it was asked: why not just tail the<br>cdr.csv file, transform, and put it into beanstalk?  I have no<br>opinion on this, just sharing a question that was asked.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Well, that is indeed doable, however, is prone to failures and worst of all, <br>add another moving part to the system. If there is one thing I've learned <br>over time, when building large scale systems, if I can reduce the number<br>of moving parts in the system - the easier it is to debug, the easier it is ?<br>to maintain.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I believe this is a comprehensive review of technical issues with<br>the current patch.  If you haven't already please take a look at<br>the wiki [1] for instructions on updating this review (instead of<br>abandoning and creating a new one).</p><p style="white-space: pre-wrap; word-wrap: break-word;">[1] https://wiki.asterisk.org/wiki/display/AST/Gerrit+Usage#GerritUsage-UpdatingaReview</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'll work the technical issues detailed below, I noticed that some of them <br>are related to me basing my work on the cdr_manager.c module, which is <br>most probably why I have some left overs in there.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6819">change 6819</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6819"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I5fe4089a34ab3b39230786d9bbfddafa56715f48 </div>
<div style="display:none"> Gerrit-Change-Number: 6819 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Nir Simionovich (GreenfieldTech - Israel) <nirs@greenfieldtech.net> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Nir Simionovich (GreenfieldTech - Israel) <nirs@greenfieldtech.net> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 17 Oct 2017 18:16:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>