cli icon indicating copy to clipboard operation
cli copied to clipboard

Timeout when trying to get diff of two files

Open calexander3 opened this issue 1 year ago • 10 comments

Hi there. I am getting a timeout when trying to compare these two files: current_api.json master_api.json

calexander3 avatar Jan 03 '24 16:01 calexander3

It was working very reliably until Dec 14th at 3:17PM Eastern. After that, only a very small amount have succeeded. We send a hand full of requests per work day. Here are those files: current_api-12-14.json master_api-12-14.json

There is about 115k size difference between the working set and the failing set

calexander3 avatar Jan 04 '24 21:01 calexander3

We were able to trim some things for the purposes of the comparison. Less than ideal but it looks to be working again. But its REALLY close to the timeout.

current_api-1-5.json

calexander3 avatar Jan 05 '24 16:01 calexander3

It started happening again. Here is the latest file: current_api-1-9.json

calexander3 avatar Jan 09 '24 23:01 calexander3

Hi @calexander3,

Thanks a lot for sharing some many details. We do have a long standing issue on our side for API documents that extensively use references ($ref). We are working actively on this to improve the support of references which should help (a lot) in terms of performance (and prevent from reaching the timeout limit that we have set to 30s and which is already way too high!).

Unfortunately I don't have a quick answer / solution for you right now but I'll be sure to let you know here when we release the ongoing work to improve reference management on our side.

Please hang tight :blush:

paulRbr avatar Jan 11 '24 15:01 paulRbr

That's very helpful information! I noticed we had some extra mime types that we don't need and those all container $ref. I removed them and it is around 20 seconds now

calexander3 avatar Jan 11 '24 16:01 calexander3

@paulRbr Here are the two files that seemed to be timing out from my standpoint.
ApplicationServicesApi.json Branch_ApplicationServicesApi.json

blemke1971 avatar Mar 15 '24 13:03 blemke1971

@paulRbr Is there any update on this issue? We started seeing this issue more frequently now!

praveenm-symplr avatar Apr 03 '24 14:04 praveenm-symplr

Hi @praveenm-symplr,

Sorry for the long response time. We will release some important changes next week which should improve the timeouts. I will make sure to ping you folks on this issue as soon as our improvements are deployed in prod.

Hang tight :hugs:

paulRbr avatar Apr 13 '24 18:04 paulRbr

@paulRbr Do we have any update on this?

praveenm-symplr avatar Jul 22 '24 17:07 praveenm-symplr

Hi @praveenm-symplr,

Are you using the CLI to do public diffs? Or are you using it as an authenticated user of Bump.sh? We are still having some performance issues for API documents with a lot of references ($ref) so that might be your case too.

Would you mind sending us an email to [email protected] with the two API documents you are trying to diff which end up in timeout please? This will help us analyze the issue and get back to you faster.

Thanks!

paulRbr avatar Jul 23 '24 07:07 paulRbr