[ 
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)

Reply via email to