You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by DylanVolz <gi...@git.apache.org> on 2017/07/27 22:30:02 UTC

[GitHub] incubator-trafficcontrol pull request #763: refactor logic to check for dupl...

GitHub user DylanVolz opened a pull request:

    https://github.com/apache/incubator-trafficcontrol/pull/763

    refactor logic to check for duplicate regexes in a cdn to reduce db IO time

    DS updates were taking around 25 seconds and failing if they had a second host regex added. With a large number of DSes.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DylanVolz/incubator-trafficcontrol fix-ds-update-regex-check

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-trafficcontrol/pull/763.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #763
    
----
commit 77a81c3572632d0bd917a47e092845e0ea436e53
Author: Dylan Volz <dy...@comcast.com>
Date:   2017-07-27T22:28:09Z

    this refactors the logic to check for duplicate regexes in a cdn to drop db IO time from ~20 secs

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafficcontrol issue #763: refactor logic to check for duplicate r...

Posted by DylanVolz <gi...@git.apache.org>.
Github user DylanVolz commented on the issue:

    https://github.com/apache/incubator-trafficcontrol/pull/763
  
    WIP - Enhancing testing as the conflicting regex check was never tested.



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafficcontrol pull request #763: [TC-497] refactor logic to check...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-trafficcontrol/pull/763


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---