[GitHub] ignite pull request #5051: IGNITE-6811: Allowed empty batch in JDBC thin dri...

2018-10-25 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/ignite/pull/5051


---


[GitHub] ignite pull request #5051: IGNITE-6811: Allowed empty batch in JDBC thin dri...

2018-10-22 Thread pavel-kuznetsov
GitHub user pavel-kuznetsov opened a pull request:

https://github.com/apache/ignite/pull/5051

IGNITE-6811: Allowed empty batch in JDBC thin driver.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gridgain/apache-ignite ignite-6811

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/ignite/pull/5051.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #5051


commit 7cd8faa9280c9708989604f17c1859f25450a6b5
Author: Pavel Kuznetsov 
Date:   2018-10-22T14:30:01Z

ignite-6811: Added tests that reproduce the bug.

When streaming is OFF we can't perform empty batch neither for Statement
nor for PreparedStatement.

commit 7cd9dd1b250655b19dafd95ad54ced751ab6f3c9
Author: Pavel Kuznetsov 
Date:   2018-10-22T15:19:38Z

ignite-6811: Fixed empty batch.

Now if empty batch is executed, no error is thrown. Empty updates counts
array is returned. This is done due to usability reasons.

commit 6ed512e24f3e0184f1ad9236dbe2e518942324c4
Author: Pavel Kuznetsov 
Date:   2018-10-22T15:53:21Z

Merge remote-tracking branch 'origin/master' into ignite-6811




---