[jira] [Comment Edited] (IGNITE-5298) .NET: DML update via LINQ

2018-03-06 Thread Pavel Tupitsyn (JIRA) 

[ 
https://issues.apache.org/jira/browse/IGNITE-5298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16387909#comment-16387909
 ] 

Pavel Tupitsyn edited comment on IGNITE-5298 at 3/6/18 3:04 PM:


[~GuruStron] looks good to me in general, awesome changes!
* Please split the test into multiple, it is too huge
* Make sure all class members have XMLDoc comments
* Some code lines exceed 120 char limit
* Why rename {{parenCount}}  to {{parentCount}} in {{CacheQueryModelVisitor}}?


was (Author: ptupitsyn):
[~GuruStron] looks good to me in general, awesome changes!
* Please split the test into multiple, it is too huge
* Make sure all class members have XMLDoc comments
* Why rename {{parenCount}}  to {{parentCount}} in {{CacheQueryModelVisitor}}?

> .NET: DML update via LINQ
> -
>
> Key: IGNITE-5298
> URL: https://issues.apache.org/jira/browse/IGNITE-5298
> Project: Ignite
>  Issue Type: New Feature
>  Components: platforms
>Affects Versions: 2.1
>Reporter: Pavel Tupitsyn
>Assignee: Pavel Tupitsyn
>Priority: Major
>  Labels: .NET, LINQ, important
>
> Bulk update with LINQ:
> {code}
> var persons = ignite.GetCache("persons").AsCacheQueryable();
> int affectedRows = persons.Where(x => x.Key > 10).UpdateAll(x => 
> x.Value.OrgId = 7);
> {code}
> See bulk delete with {{RemoveAll}}, IGNITE-4904.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Comment Edited] (IGNITE-5298) .NET: DML update via LINQ

2018-03-05 Thread Sergey Stronchinskiy (JIRA) 

[ 
https://issues.apache.org/jira/browse/IGNITE-5298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16385806#comment-16385806
 ] 

Sergey Stronchinskiy edited comment on IGNITE-5298 at 3/5/18 8:50 AM:
--

Hi, [~ptupitsyn] 

Can you review this?

[Platform 
tests|https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_IgnitePlatformNet_IgniteTests24Java8=pull%2F3599%2Fmerge=buildTypeStatusDiv]

[Inspections|https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_IgnitePlatformNetInspections_IgniteTests24Java8=pull%2F3599%2Fmerge=buildTypeStatusDiv]


was (Author: gurustron):
Hi, [~ptupitsyn],

 

Can you review this?

[Platform 
tests|https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_IgnitePlatformNet_IgniteTests24Java8=pull%2F3599%2Fmerge=buildTypeStatusDiv]

[Inspections|https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_IgnitePlatformNetInspections_IgniteTests24Java8=pull%2F3599%2Fmerge=buildTypeStatusDiv]

> .NET: DML update via LINQ
> -
>
> Key: IGNITE-5298
> URL: https://issues.apache.org/jira/browse/IGNITE-5298
> Project: Ignite
>  Issue Type: New Feature
>  Components: platforms
>Affects Versions: 2.1
>Reporter: Pavel Tupitsyn
>Assignee: Pavel Tupitsyn
>Priority: Major
>  Labels: .NET, LINQ, important
>
> Bulk update with LINQ:
> {code}
> var persons = ignite.GetCache("persons").AsCacheQueryable();
> int affectedRows = persons.Where(x => x.Key > 10).UpdateAll(x => 
> x.Value.OrgId = 7);
> {code}
> See bulk delete with {{RemoveAll}}, IGNITE-4904.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)