<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 5: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">(10 comments)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Joshua,</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thank you for your kind reviewing.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I not clear about this one.</p><p style="white-space: pre-wrap; word-wrap: break-word;">>I think topic names ultimately should be unique and descriptive themselves. To that end I'm >going to be auditing the codebase and improving this for topics and subscriptions. I think >the cleanest option is probably to have this code do a check for a topic of the same name >when creating, and if one exists then to append a unique part to the end of the passed in >name. Example:<br>><br>>A topic named "pjsip" is created.<br>>Another topic named "pjsip" is created but the code appends some unique data turning it into >"pjsip-349d93".<br>><br>>This guarantees the topic names are unique, and as the topic names are not actually used by >the API itself - whether the name passed in is used or not doesn't actually matter.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Because if there's duplicated name is already, I think it should return fail or created one in general terms of the create() function's behaviour.</p><p style="white-space: pre-wrap; word-wrap: break-word;">And, adding the unique part is not clear to usage. Every time, if the user creating new topic, they have to check the returned topic's name and have to think about why it has a different name in case of duplication.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/11013">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/11013">change 11013</a>. To unsubscribe, or for help writing mail filters, 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/c/asterisk/+/11013"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ie86d125d2966f93de74ee00f47ae6fbc8c081c5f </div>
<div style="display:none"> Gerrit-Change-Number: 11013 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 07 Mar 2019 12:08:13 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>