expose limit to be configurable from UI / command line

[lukass77]

saw that on 1.39.0 the default limit is 100 for sort by
can we configure the default limit to be smaller .. so on the first query hit ..
query will not process 100 entries by default ..
default will be 100 , but in case limit value is provided it will overwrite the default value with the provided value

I asume smaller limit can improve performance ..in case user need smaller limit

[adrianmroz-allegro]

Please note few things:

1. The '100' is not a hardcoded value. Default limit for Splits is chosen based on Dimension available limits.
2. Dimension limits are already customisable - each dimension can have different limits.
3. Visualisations can affect limits so it is not set in stone.

It would be impossible to create one setting for this. And I'm afraid it will be very hard to understand what it really does. I much prefer to avoid unnecessary configurations - they have very high maintenance cost.

[lukass77]

With regard to point number 2 , where it can be customized ?

…

On Thu, 16 Mar 2023, 18:30 Adrian Mróź, ***@***.***> wrote: Please note few things: 1. The '100' is not a hardcoded value. Default limit for Splits is chosen based on Dimension available limits. 2. Dimension limits are already customisable - each dimension can have different limits. 3. Visualisations can affect limits so it is not set in stone. It would be impossible to create one setting for this. And I'm afraid it will be very hard to understand what it really does. I much prefer to avoid unnecessary configurations - they have very high maintenance cost. — Reply to this email directly, view it on GitHub <#1050 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXXXNTANGJGZJOMKSNIV6LW4M5YVANCNFSM6AAAAAAV2BB2GI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[adrianmroz]

It's just property limits in dimension configuration.

I see it's not in the documentation. I'll add it tomorrow!

[lukass77]

Hey any update on this , documentation change ?

[adrianmroz-allegro]

Documentation added in #1058