[
https://issues.apache.org/jira/browse/IGNITE-5468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16119987#comment-16119987
]
Andrew Mashenkov commented on IGNITE-5468:
------------------------------------------
PR looks fine for me.
One thing bother me I've no idea to make better.
This is how we validate IGNITE_SQL_MERGE_TABLE_MAX_SIZE parameter and fallback
to default if there is any issue.
Code looks too complex.
[~avinogradov] please take a look at it and merge if its ok for you.
> Need to set IGNITE_SQL_MERGE_TABLE_MAX_SIZE on per query basis
> --------------------------------------------------------------
>
> Key: IGNITE-5468
> URL: https://issues.apache.org/jira/browse/IGNITE-5468
> Project: Ignite
> Issue Type: Improvement
> Components: sql
> Affects Versions: 2.0
> Reporter: Yakov Zhdanov
> Assignee: Vitaliy Biryukov
> Priority: Critical
> Fix For: 2.2
>
>
> Currently this property can be set via sys property only thus changing it
> will require restart of the cluster. I think it is better to set it on
> perquery basis with some default that is configured via cache configuration.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)