We just upgraded to Stash 3.0.4. We have a repository that contains ~45000 files and it would be very nice to compare its branches. However, the diff shows the above message and only shows the first 1000 files. Which config parameter do I need to set to increase this limit?
I tried page.max.changes=50000, but this does not seem to be the right one.
(I know this can increase the RAM usage, the server should be able to handle it.)
This is a bug/feature request, will be fixed by Atlassian. (See discussion below.)
Hi Balazs,
I found a document describing this behavior and presenting a resolution to it: https://confluence.atlassian.com/display/STASHKB/Pull+request+is+too+large+to+render
It contains an additional step to be done for Stash 1.3.0 or older. Please check if this is your case and if this document is helpful.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
I am going to try soon.
Side note: this is a bug that should be fixed. I should never need to modify .js files because they are overwritten every time Stash is updated. (The config parameter should work by itself and the .js file should use the value from here instead of redefining it.)
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
I tried, no effect. BTW this extra step is for older versions than 1.3.0 and as said above, we have 3.0.4. Did I find a bug?
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
Any further suggestions or should I file a bug report?
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
Hi Balazs,
There's nothing you can do locally (except modify the js file). I would raise a bug - although it make take a few releases to trickle through to the public.
EDIT: Actually looking at it further I think you can blame me - the value is not being passed from (the correct) global setting 'page.max.changes' to the compare-diff.js JS file (which I wrote). I'll try to ping someone on the Stash team - it's a trivial fix.
Cheers,
Charles
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
Hi,
Good to hear you already know a fix. Should I create a bug for it or do you just fix it?
Thanks,
Balazs
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
Sadly (for us), Charles doesn't work on Stash anymore. He's just such a good engineer and a great guy that he's still taking ownership of his features. :)
I've created https://jira.atlassian.com/browse/STASH-5079for us to look at it. It should indeed be a trivial fix.
Cheers,
Adam
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
I created the following issue for this, with a fix on the way: https://jira.atlassian.com/browse/STASH-5080
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
Now there are two issues for it.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
That was my fault, sorry about that. We closed one as a duplicate.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
Online forums and learning are now in one easy-to-use experience.
By continuing, you accept the updated Community Terms of Use and acknowledge the Privacy Policy. Your public name, photo, and achievements may be publicly visible and available in search engines.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.