pull-request-notifier-for-bitbucket icon indicating copy to clipboard operation
pull-request-notifier-for-bitbucket copied to clipboard
verified publisher icon tomasbjerre

Trigger build button displays slower as PR's increase

Open pemercer opened this issue 8 years ago • 9 comments

Bitbucket 4.14.5 PR Notifier 2.67 Bitbucket DC 5K users

the button definition is retrieved from the REST endpoint

https:///bitbucket/rest/prnfb-admin/1.0/settings/buttons/repository/{repo-num}/pullrequest/{pr-num}

some old PRs do show the button successfully (e.g., PRs 100 and 200). In these cases, the endpoint returns a response in 6-8 seconds in my trials:

https:///bitbucket/rest/prnfb-admin/1.0/settings/buttons/repository/6940/pullrequest/100 https:///bitbucket/rest/prnfb-admin/1.0/settings/buttons/repository/6940/pullrequest/200

Now if you try the same thing for one of the later PRs (I’m using PR 578), it takes 276-287 seconds in my trials to respond.

pemercer avatar Sep 27 '17 03:09 pemercer

Perhaps same as https://github.com/tomasbjerre/pull-request-notifier-for-bitbucket/issues/237

tomasbjerre avatar Sep 27 '17 07:09 tomasbjerre

Can #237 be ported to BBS 4.x?

pemercer avatar Sep 27 '17 17:09 pemercer

Yes it can. But Atlassian is making it hard to maintain several major versions so it is not a priority for me.

tomasbjerre avatar Sep 28 '17 04:09 tomasbjerre

Ok We will go ahead and move to BB 5.x this weekend and see if the new version helps.

pemercer avatar Sep 28 '17 14:09 pemercer

BB DC server is now upgraded to BB 5.3.2, other plugin that was filling the event queue has been removed. PR Notifier version 3.15 is installed Still seeing the slow response time for the configured button to show up.

pemercer avatar Oct 30 '17 17:10 pemercer

Is there a planned fix date, or additional information I can provide to help with debug?

pemercer avatar Nov 14 '17 19:11 pemercer

No not right now.

tomasbjerre avatar Nov 15 '17 05:11 tomasbjerre

We will be moving to 5.6.1 in a few weeks, will there be a newer version of the plugin that may have this issue fixed by then?

pemercer avatar Dec 14 '17 17:12 pemercer

I dont think so.

Den 14 dec. 2017 18:12 skrev "pemercer" [email protected]:

We will be moving to 5.6.1 in a few weeks, will there be a newer version of the plugin that may have this issue fixed by then?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/tomasbjerre/pull-request-notifier-for-bitbucket/issues/250#issuecomment-351776073, or mute the thread https://github.com/notifications/unsubscribe-auth/AAa1EyD84WdwRdA569BZ42wkvzZKrebXks5tAVb1gaJpZM4PlLss .

tomasbjerre avatar Dec 14 '17 18:12 tomasbjerre