[GitHub] ignite pull request #2533: IGNITE-6122: Propagate SqlFieldsQuery.lazy proper...

2017-08-29 Thread isapego
Github user isapego closed the pull request at: https://github.com/apache/ignite/pull/2533 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Re: Troubles with BigDecimal and BigInteger - maybe disable entirely?

2017-08-29 Thread Ilya Kasnacheev
Hello Andrey. I'm not sure that BigDecimal makes for a passable SQL DECIMAL. Somebody have to look at the standard sooner or later. BigDecimal works as values as well, and BigInteger don't work, too. I've created this remedy pull request and now am waiting for tests:

[GitHub] ignite pull request #2543: IGNITE-6119: Added 'lazy' flag to ODBC

2017-08-29 Thread isapego
GitHub user isapego opened a pull request: https://github.com/apache/ignite/pull/2543 IGNITE-6119: Added 'lazy' flag to ODBC You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-6119 Alternatively you

[jira] [Created] (IGNITE-6219) IgniteCache#loadCache executes local load in caller thread

2017-08-29 Thread Valentin Kulichenko (JIRA)
Valentin Kulichenko created IGNITE-6219: --- Summary: IgniteCache#loadCache executes local load in caller thread Key: IGNITE-6219 URL: https://issues.apache.org/jira/browse/IGNITE-6219 Project:

[GitHub] ignite pull request #2544: Tx recovery fix

2017-08-29 Thread agura
GitHub user agura opened a pull request: https://github.com/apache/ignite/pull/2544 Tx recovery fix You can merge this pull request into a Git repository by running: $ git pull https://github.com/agura/incubator-ignite tx-recovery-fix Alternatively you can review and apply

[jira] [Created] (IGNITE-6218) Document logging capabilities of Ignite

2017-08-29 Thread Denis Magda (JIRA)
Denis Magda created IGNITE-6218: --- Summary: Document logging capabilities of Ignite Key: IGNITE-6218 URL: https://issues.apache.org/jira/browse/IGNITE-6218 Project: Ignite Issue Type: Task

[GitHub] ignite pull request #2536: IGNITE-6210 Size of the checkpoint buffer is limi...

2017-08-29 Thread DmitriyGovorukhin
GitHub user DmitriyGovorukhin opened a pull request: https://github.com/apache/ignite/pull/2536 IGNITE-6210 Size of the checkpoint buffer is limited You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite

sql documentation

2017-08-29 Thread Serge Puchnin
Hi ! As I can see, there is some room for documentation improvement. As a sample of documentation, I've prepared a description of "CREATE INDEX" statement. What’s your opinion? What can be improved? What should be added? Thanks a lot! --- Serge

[jira] [Created] (IGNITE-6209) Build Ignite.NET NuGet packages for Apache-Ignite release on CI

2017-08-29 Thread Oleg Ostanin (JIRA)
Oleg Ostanin created IGNITE-6209: Summary: Build Ignite.NET NuGet packages for Apache-Ignite release on CI Key: IGNITE-6209 URL: https://issues.apache.org/jira/browse/IGNITE-6209 Project: Ignite

[jira] [Created] (IGNITE-6208) SQL: Add documentation for lazy query execution

2017-08-29 Thread Vladimir Ozerov (JIRA)
Vladimir Ozerov created IGNITE-6208: --- Summary: SQL: Add documentation for lazy query execution Key: IGNITE-6208 URL: https://issues.apache.org/jira/browse/IGNITE-6208 Project: Ignite Issue

[jira] [Created] (IGNITE-6210) inefficient memory consumption for checkpoint buffer

2017-08-29 Thread Dmitriy Govorukhin (JIRA)
Dmitriy Govorukhin created IGNITE-6210: -- Summary: inefficient memory consumption for checkpoint buffer Key: IGNITE-6210 URL: https://issues.apache.org/jira/browse/IGNITE-6210 Project: Ignite

Re: ContinuousQueryWithTransformer implementation questions - 2

2017-08-29 Thread Anton Vinogradov
Nikolay, > 2. What behavior is expected if transformer throws exception for some event? I see following options: Client should be notified, I vote for > Introduce special callback. onTransformError? On Tue, Aug 29, 2017 at 1:36 PM, Nikolay Izhikov wrote: > Hello,

[GitHub] ignite pull request #2535: IGNITE-6182

2017-08-29 Thread gvvinblade
GitHub user gvvinblade opened a pull request: https://github.com/apache/ignite/pull/2535 IGNITE-6182 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-6182 Alternatively you can review and apply

[jira] [Created] (IGNITE-6211) ODBC: SQLBindParameter should not unbind parameter if the ParameterValuePtr is NULL

2017-08-29 Thread Igor Sapego (JIRA)
Igor Sapego created IGNITE-6211: --- Summary: ODBC: SQLBindParameter should not unbind parameter if the ParameterValuePtr is NULL Key: IGNITE-6211 URL: https://issues.apache.org/jira/browse/IGNITE-6211

Re: ContinuousQueryWithTransformer implementation questions - 2

2017-08-29 Thread Yakov Zhdanov
I don't like the idea of having separate class, but it seems to be the only way as there are too many API and generics differences. Nikolay, I would also suggest you extract some super class for continuous query. It will help to avoid code duplicates. As far as remote transformer failure - we

ContinuousQueryWithTransformer implementation questions - 2

2017-08-29 Thread Nikolay Izhikov
Hello, Igniters. I'm working on IGNITE-425 [1] issue. Text of issue: === Currently if updated entry passes the filter, it is sent to node initiated the query entirely. It would be good to provide user with the ability to transform entry and, for example, select only fields that are

[GitHub] ignite pull request #2491: IGNITE-6118 JDBC Thin: Propagate "lazy" flag

2017-08-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/2491 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Override cache name created with CREATE TABLE

2017-08-29 Thread Denis Magda
Igniters, That’s one more feedback about CREATE TABLE usage in practice. The command automatically creates an IgniteCache naming it SQL_PUBLIC_{TABLE}. So, if a Person table is created you’ll have SQL_PUBLIC_PERSON cache in the cluster. Honestly, if you keep to SQL APIs the cache name won’t

Re: CREATE TABLE usage from Java API, .NET, C++

2017-08-29 Thread Valentin Kulichenko
Ideally, SQL API has to be completely decoupled from cache API. Otherwise we will keep getting issues like this. let's introduce top level API (IgniteSql?) and add everything there. -Val On Tue, Aug 29, 2017 at 7:47 PM Denis Magda wrote: > Igniters, > > Not sure we

Re: Adding sqlline tool to Apache Ignite project

2017-08-29 Thread Denis Magda
Igniters, Let me introduce Julian Hyde [1], creator of SQLLine tool and our Apache mate, Julian, Please grant that Apache Ignite community a permission to include SQLLine [2] it in every Ignite deliverable (source, binary). It’s planned to suggest the tool as a default command line SQL

New PMC: Pavel Tupitsyn

2017-08-29 Thread Denis Magda
The Project Management Committee (PMC) for Apache Ignite has invited Pavel Tupitsyn to become a PMC member and we are pleased to announce that he has accepted the role. Being a PMC member enables assistance with the management and to guide the direction of the project. Pavel, is a veteran of

CREATE TABLE usage from Java API, .NET, C++

2017-08-29 Thread Denis Magda
Igniters, Not sure we discussed this before, so let me start a new thread. It’s claimed the command is supported from native Java, .NET, C++ APIs but I had hard time trying to use it from there. Imagine this simple statement to be called from Java source code: SqlFieldsQuery query = new

[jira] [Created] (IGNITE-6212) Assertion error: Invalid node2part

2017-08-29 Thread Ilya Lantukh (JIRA)
Ilya Lantukh created IGNITE-6212: Summary: Assertion error: Invalid node2part Key: IGNITE-6212 URL: https://issues.apache.org/jira/browse/IGNITE-6212 Project: Ignite Issue Type: Bug

Re: ContinuousQueryWithTransformer implementation questions - 2

2017-08-29 Thread Yakov Zhdanov
> If filter throws exception entry would be passed to listener. this is strange. Imagine a filter that very rarely throws some runtime exception due to external or environmental reasons, but in case of normal execution filter evaluates to false. In case of error entry is passed to a local

[GitHub] ignite pull request #2537: IGNITE-6193 ML profile is missing in 2.1 binary r...

2017-08-29 Thread oignatenko
GitHub user oignatenko opened a pull request: https://github.com/apache/ignite/pull/2537 IGNITE-6193 ML profile is missing in 2.1 binary release - verified with diffs overview and trial local build You can merge this pull request into a Git repository by running: $ git pull

[GitHub] ignite pull request #2448: IGNITE-6050

2017-08-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/2448 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Re: ContinuousQueryWithTransformer implementation questions - 2

2017-08-29 Thread Nikolay Izhikov
Hi, Yakov. If filter throws exception entry would be passed to listener. > Nikolay, I would also suggest you extract some super class for continuous > query. It will help to avoid code duplicates. Yes, I will do this after reaching consensus on API changes. 29.08.2017 14:04, Yakov Zhdanov

Re: ContinuousQueryWithTransformer implementation questions - 2

2017-08-29 Thread Nikolay Izhikov
Yakov. I think exception equals `true` is intended behavior. Filter evaluation implementation from master - [1] Test from master to check filter exception(without explicit asserts checking listeners call) - [2] Here is my quick test with asserts on listener call after filter exception: ```

[GitHub] ignite pull request #2538: IGNITE-6212 : Fixed assertion error for invalid n...

2017-08-29 Thread ilantukh
GitHub user ilantukh opened a pull request: https://github.com/apache/ignite/pull/2538 IGNITE-6212 : Fixed assertion error for invalid node2part. You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-6212

Re: ContinuousQueryWithTransformer implementation questions - 2

2017-08-29 Thread Yakov Zhdanov
Igniters, Does anyone else see potential issues on user side with current approach? Sam, is this JCache requirement? --Yakov 2017-08-29 15:16 GMT+03:00 Nikolay Izhikov : > Yakov. > > I think exception equals `true` is intended behavior. > > Filter evaluation

[jira] [Created] (IGNITE-6213) Unexpected setting local deployment owner anyone node

2017-08-29 Thread Vladislav Pyatkov (JIRA)
Vladislav Pyatkov created IGNITE-6213: - Summary: Unexpected setting local deployment owner anyone node Key: IGNITE-6213 URL: https://issues.apache.org/jira/browse/IGNITE-6213 Project: Ignite

[jira] [Created] (IGNITE-6214) Client hangs when executing SQL updates

2017-08-29 Thread Denis Mekhanikov (JIRA)
Denis Mekhanikov created IGNITE-6214: Summary: Client hangs when executing SQL updates Key: IGNITE-6214 URL: https://issues.apache.org/jira/browse/IGNITE-6214 Project: Ignite Issue Type:

[GitHub] ignite pull request #2446: IGNITE-4642 Support enforceJoinOrder flag for JDB...

2017-08-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/2446 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Re: Plugins in tests

2017-08-29 Thread Pavel Tupitsyn
I've replied in JIRA. Yakov, please have a look as well. On Wed, Aug 23, 2017 at 6:44 PM, Vyacheslav Daradur wrote: > Hi, > > >> ticket filed: https://issues.apache.org/jira/browse/IGNITE-5879 > I've done it, could someone review it? > > 2017-07-31 11:13 GMT+03:00 Pavel

guard() and checkClusterState() checks in Ignite Kernal

2017-08-29 Thread Ilya Kasnacheev
Dear Igniters, I would like to know if guard() and checkClusterState() methods are always called consistently in IgniteKernal. Let's look at the list: Out of cluster(), localNode(), compute(), message(), events(), executorService(), services() only the last one is guarded by checkClusterState()

[GitHub] ignite pull request #2511: IGNITE-6178 Make CheckpointWriteOrder.SEQUENTIAL ...

2017-08-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/2511 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Troubles with BigDecimal and BigInteger - maybe disable entirely?

2017-08-29 Thread Ilya Kasnacheev
Hello fellow Igniters, We have troubles with BigDecimal and BigInteger types used as cache keys and then referenced in SQL. As far as I see, our implementation of SQL considers BigDecimal to be SQL's DECIMAL type. But semantics of BigDecimal are different. BigDecimal("4.2") not equals to

[GitHub] ignite pull request #2539: Ignite 5817 x

2017-08-29 Thread oleg-ostanin
GitHub user oleg-ostanin opened a pull request: https://github.com/apache/ignite/pull/2539 Ignite 5817 x You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-5817-x Alternatively you can review and

[GitHub] ignite pull request #2540: Ignite 2.1.3.b5

2017-08-29 Thread ntikhonov
GitHub user ntikhonov opened a pull request: https://github.com/apache/ignite/pull/2540 Ignite 2.1.3.b5 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-2.1.3.b5 Alternatively you can review and

[jira] [Created] (IGNITE-6216) Add CheckpointWriteOrder enum in .NET persistent store configuration

2017-08-29 Thread Ivan Rakov (JIRA)
Ivan Rakov created IGNITE-6216: -- Summary: Add CheckpointWriteOrder enum in .NET persistent store configuration Key: IGNITE-6216 URL: https://issues.apache.org/jira/browse/IGNITE-6216 Project: Ignite

[GitHub] ignite pull request #2541: IGNITE-5620

2017-08-29 Thread alexpaschenko
GitHub user alexpaschenko opened a pull request: https://github.com/apache/ignite/pull/2541 IGNITE-5620 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-5620 Alternatively you can review and apply

Re: Troubles with BigDecimal and BigInteger - maybe disable entirely?

2017-08-29 Thread Andrey Mashenkov
Ilya, Ignite doesn't support scale\precision constrains for Decimal type, so H2 treat them as Decimals with highest precision. It is different issue as BigDecimal keys works, but with limitation. However, BigInteger keys looks like doesn't work at all. Disable having BigDecimal & BigInteger

[jira] [Created] (IGNITE-6217) Add benchmark to compare JDBC drivers and native SQL execution

2017-08-29 Thread Taras Ledkov (JIRA)
Taras Ledkov created IGNITE-6217: Summary: Add benchmark to compare JDBC drivers and native SQL execution Key: IGNITE-6217 URL: https://issues.apache.org/jira/browse/IGNITE-6217 Project: Ignite

[GitHub] ignite pull request #2542: IGNITE-5905 .NET: Thin client: cache.Get for prim...

2017-08-29 Thread ptupitsyn
GitHub user ptupitsyn opened a pull request: https://github.com/apache/ignite/pull/2542 IGNITE-5905 .NET: Thin client: cache.Get for primitives You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-5905

Re: Ignite 2.1.0 - Thread deadlock on DefaultSingletonBeanRegistry.getSingleton

2017-08-29 Thread Denis Magda
Igor, Could you share a test to reproduce the issue? — Denis > On Aug 29, 2017, at 3:59 AM, igor.tanackovic > wrote: > > Upgraded Ignite from 2.0.0 to 2.1.0 and ran into a thread deadlock on Ignite > startup. I noticed the deadlock once I added CacheStore into a

Re: sql documentation

2017-08-29 Thread Denis Magda
Serge, The attachment is missing. Try to share it via google drive or other storage. — Denis > On Aug 29, 2017, at 2:38 AM, Serge Puchnin wrote: > > Hi ! > > As I can see, there is some room for documentation improvement. > As a sample of documentation, I've

[GitHub] ignite pull request #2545: For testing

2017-08-29 Thread ilantukh
GitHub user ilantukh opened a pull request: https://github.com/apache/ignite/pull/2545 For testing You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-2.1.4-6212 Alternatively you can review and apply