[jira] [Commented] (TC-135) /api/:version/servers/hostname/:name/details spawns n+1 queries

2017-02-07 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/TC-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856213#comment-15856213
 ] 

ASF GitHub Bot commented on TC-135:
---

Github user asfgit closed the pull request at:

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


> /api/:version/servers/hostname/:name/details spawns n+1 queries
> ---
>
> Key: TC-135
> URL: https://issues.apache.org/jira/browse/TC-135
> Project: Traffic Control
>  Issue Type: Improvement
>  Components: Traffic Ops, Traffic Ops API
>Affects Versions: 2.0.0
>Reporter: Jeremy Mitchell
>Assignee: Jeremy Mitchell
>Priority: Minor
> Fix For: 2.0.0
>
>
> The query behind this api route spawn sequential n+1 queries which degrades 
> performance.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (TC-135) /api/:version/servers/hostname/:name/details spawns n+1 queries

2017-02-06 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/TC-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15854779#comment-15854779
 ] 

ASF GitHub Bot commented on TC-135:
---

GitHub user mitchell852 opened a pull request:

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

[TC-135] - joins ds_server on ds table to avoid n+1 queries and improve 
route pe…

…rformance 10x at least

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

$ git pull https://github.com/mitchell852/incubator-trafficcontrol tc-135

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

https://github.com/apache/incubator-trafficcontrol/pull/254.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 #254


commit 85bbf7baa339b5dd996f60d63e5709e3812f7629
Author: Jeremy Mitchell 
Date:   2017-02-06T21:30:18Z

joins ds_server on ds table to avoid n+1 queries and improve route 
performance 10x at least




> /api/:version/servers/hostname/:name/details spawns n+1 queries
> ---
>
> Key: TC-135
> URL: https://issues.apache.org/jira/browse/TC-135
> Project: Traffic Control
>  Issue Type: Improvement
>  Components: Traffic Ops, Traffic Ops API
>Affects Versions: 2.0.0
>Reporter: Jeremy Mitchell
>Assignee: Jeremy Mitchell
>Priority: Minor
> Fix For: 2.0.0
>
>
> The query behind this api route spawn sequential n+1 queries which degrades 
> performance.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)