WebConsole configuration wizard doesn't properly handle multiple RDBMS

2017-05-26 Thread Denis Magda
Alex K, Andrey N, Imagine a use case when cache_A needs to persist data in database_A while cache_B has to be connected with database_B as a part of a single application. WebConsole allows importing a schema from both databases but: • in a final Spring XML configuration there will be onl

[jira] [Created] (IGNITE-5303) WebConsole configuration wizard doesn't properly handle multiple RDBMS

2017-05-26 Thread Denis Magda (JIRA)
Denis Magda created IGNITE-5303: --- Summary: WebConsole configuration wizard doesn't properly handle multiple RDBMS Key: IGNITE-5303 URL: https://issues.apache.org/jira/browse/IGNITE-5303 Project: Ignite

Re: New blog post: ADO.NET Ignite Cache Store

2017-05-26 Thread Denis Magda
Great, Pavel! You rock! Prachi, could you send the post to DZone and publish on Ignite’s page? — Denis > On May 26, 2017, at 10:06 AM, Pavel Tupitsyn wrote: > > Short writeup with example code, inspired by user questions: > > ADO.NET + Binary Mode Cache Store > https://ptupi

[RESULT][VOTE] Accept Contribution of Ignite Persistent Store

2017-05-26 Thread Denis Magda
Igniters, Thanks to all for you for the participating in the VOTE: http://apache-ignite-developers.2346864.n4.nabble.com/VOTE-Accept-Contribution-of-Ignite-Persistent-Store-td17896.html

[RESULT][VOTE] Accept Contribution of Ignite Persistent Store

2017-05-26 Thread Denis Magda
Igniters, Thanks to all for you for the participating in the VOTE: http://apache-ignite-developers.2346864.n4.nabble.com/VOTE-Accept-Contribution-of-Ignite-Persistent-Store-td17896.html The VOTE is closed with nine “+1” votes: - Semyon Boikov (binding) - Valentin Kulichenko (binding) - Sergi Vl

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
Well, merged the fix to the branch with the donation and closed the ticket: https://issues.apache.org/jira/browse/IGNITE-5295 — Denis > On May 26, 2017, at 1:25 PM, Denis Magda wrote: > > As I see from the AffinityFunction implementations, cl

Re: Unclear configuration parameters of Persistent Store

2017-05-26 Thread Denis Magda
Alex G., > * PersistentStoreConfiguration.setWalHistorySize > * PersistentStoreConfiguration.setWalArchivePath I could figure out the goal of these two parameters below referring to the store architecture guide [1] and updated the Java docs. > PersistentStoreConfiguration.setLockWaitTime Howev

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
As I see from the AffinityFunction implementations, client nodes are not used there that implies there is no way to deploy replicated and partitioned caches there. — Denis > On May 26, 2017, at 12:19 PM, Dmitriy Setrakyan wrote: > > On Fri, May 26, 2017 at 12:17 PM, Denis Magda wrote: > >>

Re: Persistent Store example fails with NPE

2017-05-26 Thread Dmitriy Setrakyan
On Fri, May 26, 2017 at 12:17 PM, Denis Magda wrote: > Local and near caches can be configured on clients. But you can still > define replicated and partitioned ones in the clients’ configuration and > they will be simply ignored. I want us to do the same for the persistent > store (simply ignore

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
Local and near caches can be configured on clients. But you can still define replicated and partitioned ones in the clients’ configuration and they will be simply ignored. I want us to do the same for the persistent store (simply ignore it if it’s defined in a client’s configuration). Sounds re

Re: Persistent Store example fails with NPE

2017-05-26 Thread Dmitriy Setrakyan
On Fri, May 26, 2017 at 11:59 AM, Denis Magda wrote: > Exactly, you can specify caches in the configuration of both clients and > servers BUT the caches will be deployed on the servers only. > This is why it’s absolutely fine to use a single configuration (with > caches defined) for all your clus

Re: Reason for disabling @QuerySqlField and @AffinityKeyMapped annotations on methods

2017-05-26 Thread Dmitriy Setrakyan
On Fri, May 26, 2017 at 12:01 PM, Vladimir Ozerov wrote: > I even think that we should remove Marshaller interface at all in future > major release and use only binary. Too many things just work without > BinaryMarshaller. We cannot afford non-binary Marshallers any more. > Agree, but let's Depr

Re: Reason for disabling @QuerySqlField and @AffinityKeyMapped annotations on methods

2017-05-26 Thread Vladimir Ozerov
I even think that we should remove Marshaller interface at all in future major release and use only binary. Too many things just work without BinaryMarshaller. We cannot afford non-binary Marshallers any more. пт, 26 мая 2017 г. в 21:49, Dmitriy Setrakyan : > On Fri, May 26, 2017 at 8:29 AM, Igor

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
Exactly, you can specify caches in the configuration of both clients and servers BUT the caches will be deployed on the servers only. This is why it’s absolutely fine to use a single configuration (with caches defined) for all your cluster nodes because Ignite can automatically figure out what

Re: Reason for disabling @QuerySqlField and @AffinityKeyMapped annotations on methods

2017-05-26 Thread Dmitriy Setrakyan
On Fri, May 26, 2017 at 8:29 AM, Igor Rudyak wrote: > If binary objects is the only option starting from Ignite 2.0 than it makes > sense. > > If we still support regular java objects, than it looks a bit confusing to > disallowing such annotations. Especially if they don't break anything. > We

Re: Persistent Store example fails with NPE

2017-05-26 Thread Dmitriy Setrakyan
Denis, I think I misunderstand the issue. How do we do it for caches? Caches are only supposed to be enabled on the servers by default. Do we require a different configuration file for clients and servers for the caches? D. On Fri, May 26, 2017 at 11:41 AM, Denis Magda wrote: > I tend to use a

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
I tend to use a single XML configuration for all my cluster nodes (servers and clients). Presently, I forced to create and maintain at least 2. This is the first concern. The second concern is if persistence store has no effect for the client node why not simply ignore it and not make the user t

Re: Persistent Store example fails with NPE

2017-05-26 Thread Dmitriy Setrakyan
On Fri, May 26, 2017 at 11:29 AM, Denis Magda wrote: > > Can we just fix the exception > > message to clearly state that client mode requires all explicit > > configuration for such and such functionality? > > I don’t like this. It will require me as a user: > > * have a separate configuration fo

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
> Can we just fix the exception > message to clearly state that client mode requires all explicit > configuration for such and such functionality? I don’t like this. It will require me as a user: * have a separate configuration for clients ALL the times. * initialize the memory bean explicitly wh

Re: AffinityKeyMapper

2017-05-26 Thread Dmitriy Setrakyan
On Fri, May 26, 2017 at 8:48 AM, Sergi Vladykin wrote: > Guys, > > As I see we did not drop AffinityKeyMapper for 2.0. > > May be lets at least deprecate it? > I think we must. Any objections?

Re: Persistent Store example fails with NPE

2017-05-26 Thread Dmitriy Setrakyan
I don't like ignoring any configuration, but I do agree that enabling persistence on the client side seems odd. Can we just fix the exception message to clearly state that client mode requires all explicit configuration for such and such functionality? D. On Fri, May 26, 2017 at 10:34 AM, Sergey

Re: Persistent Store example fails with NPE

2017-05-26 Thread Sergey Chugunov
Denis, Thanks for quick reply, I made a change and tested it with your example, everything looks fine. Pull request is open and available at [1] [1] https://github.com/apache/ignite/pull/2016 On Fri, May 26, 2017 at 8:25 PM, Denis Magda wrote: > > I would suggest on the first iteration to si

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
> I would suggest on the first iteration to simply ignore persistence > configuration on client nodes. > If client node finds persistence configuration on startup it will printout > warning message to log and proceed without persistence. +1 for this. Otherwise, it looks like a dirty hack when I ne

[GitHub] ignite pull request #2016: persistence configuration should be ignored on cl...

2017-05-26 Thread sergey-chugunov-1985
GitHub user sergey-chugunov-1985 opened a pull request: https://github.com/apache/ignite/pull/2016 persistence configuration should be ignored on client nodes You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite

New blog post: ADO.NET Ignite Cache Store

2017-05-26 Thread Pavel Tupitsyn
Short writeup with example code, inspired by user questions: ADO.NET + Binary Mode Cache Store https://ptupitsyn.github.io/Ado-Net-Cache-Store/

Re: Persistent Store example fails with NPE

2017-05-26 Thread Sergey Chugunov
Moreover, enabling persistence on client node is very risky and may lead to many tricky cases. I would suggest on the first iteration to simply ignore persistence configuration on client nodes. If client node finds persistence configuration on startup it will printout warning message to log and pr

Re: Persistent Store example fails with NPE

2017-05-26 Thread Sergey Chugunov
Denis, I see from example that you're trying to start a client node with persistence configuration specified and you're saying that Ignite must provide default MemoryConfiguration. But that contradicts with decision we all agreed on in the discussion[1]: if user wants to do something unusual on c

Re: Persistent Store example fails with NPE

2017-05-26 Thread Denis Magda
Sergey, The issue vanishes as soon as I add the following to the configuration: My point is that we just need to create a MemoryConfiguration implicitly if it’s not defined (no any warnings or extra messages). It’s a responsibility of our platform to generate a default configuratio

AffinityKeyMapper

2017-05-26 Thread Sergi Vladykin
Guys, As I see we did not drop AffinityKeyMapper for 2.0. May be lets at least deprecate it? Sergi

Re: Persistent Store example fails with NPE

2017-05-26 Thread Sergey Chugunov
Hi Denis, What is the expected behavior in that case? Bug doesn't suggest any. >From my point of view it would be better to throw an exception on startup with meaningful message than to silently create a default MemoryConfiguration under the cover. Thanks, Sergey. On Fri, May 26, 2017 at 4:00 A

Re: Reason for disabling @QuerySqlField and @AffinityKeyMapped annotations on methods

2017-05-26 Thread Igor Rudyak
If binary objects is the only option starting from Ignite 2.0 than it makes sense. If we still support regular java objects, than it looks a bit confusing to disallowing such annotations. Especially if they don't break anything. Igor On May 26, 2017 12:55 AM, "Vladimir Ozerov" wrote: Igor,

[GitHub] ignite pull request #2015: IGNITE-5087

2017-05-26 Thread NSAmelchev
GitHub user NSAmelchev opened a pull request: https://github.com/apache/ignite/pull/2015 IGNITE-5087 Fix Enum comparison fails after marshal-unmarshal with BinaryMarshaller. You can merge this pull request into a Git repository by running: $ git pull https://github.com/NSAmelch

[jira] [Created] (IGNITE-5302) Empty LOST partition may be used as OWNING after resetting lost partitions

2017-05-26 Thread Sergey Chugunov (JIRA)
Sergey Chugunov created IGNITE-5302: --- Summary: Empty LOST partition may be used as OWNING after resetting lost partitions Key: IGNITE-5302 URL: https://issues.apache.org/jira/browse/IGNITE-5302 Proj

[jira] [Created] (IGNITE-5301) JVM crushes on H2TreeIndex destroy

2017-05-26 Thread Igor Seliverstov (JIRA)
Igor Seliverstov created IGNITE-5301: Summary: JVM crushes on H2TreeIndex destroy Key: IGNITE-5301 URL: https://issues.apache.org/jira/browse/IGNITE-5301 Project: Ignite Issue Type: Bug

[jira] [Created] (IGNITE-5300) .NET: LINQ RemoveAll examples

2017-05-26 Thread Pavel Tupitsyn (JIRA)
Pavel Tupitsyn created IGNITE-5300: -- Summary: .NET: LINQ RemoveAll examples Key: IGNITE-5300 URL: https://issues.apache.org/jira/browse/IGNITE-5300 Project: Ignite Issue Type: Improvement

Re: Ignite 2.0 TeamCity Run configurations

2017-05-26 Thread Dmitry Pavlov
Igniters, I've found one more suite without tests: "Ignite Cache Failover Multi JVM" (org.apache.ignite.testsuites.IgniteCacheFailoverMJTestSuite). Currently has paused triggers, but anyway started from Ignite 2.0 snapshot dependecies. Is it required? Alex (al.psc) do you know? it is required in

[GitHub] ignite pull request #1678: Ignite 1925

2017-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/1678 --- 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 ena

[jira] [Created] (IGNITE-5299) Don't set IgniteLock as broken if lock is failoverSafe

2017-05-26 Thread Evgenii Zhuravlev (JIRA)
Evgenii Zhuravlev created IGNITE-5299: - Summary: Don't set IgniteLock as broken if lock is failoverSafe Key: IGNITE-5299 URL: https://issues.apache.org/jira/browse/IGNITE-5299 Project: Ignite

Re: IgfsPerBlockLruEvictionPolicy does not work as expected any more, what shall we do with it?

2017-05-26 Thread Ivan V.
Denis, many thanks for the info. Phrase in docs "on-heap caching is for scenarios when you do a lot of cache reads on server nodes that *work with cache entries in the binary form* or invoke cache entries' deserialization." may be contains a mistake: should it say "*work with cache entries in non-

Re: Ignite 2.0 TeamCity Run configurations

2017-05-26 Thread Dmitry Pavlov
Thank you! I've commited removal of both suites to PR https://github.com/apache/ignite/pull/2014 пт, 26 мая 2017 г. в 14:53, Vladimir Ozerov : > No, we should remote this suite. > > On Fri, May 26, 2017 at 2:48 PM, Semyon Boikov > wrote: > > > What about 'Ignite H2 Indexing' suite', it does no

Re: Ignite 2.0 TeamCity Run configurations

2017-05-26 Thread Vladimir Ozerov
No, we should remote this suite. On Fri, May 26, 2017 at 2:48 PM, Semyon Boikov wrote: > What about 'Ignite H2 Indexing' suite', it does not execute any test? > > On Fri, May 26, 2017 at 2:18 PM, Dmitry Pavlov > wrote: > > > Hi Semen, > > > > > > > > Thank you for this fix. It is correct to rep

[GitHub] ignite pull request #2014: ignite-4947: binary basic test suite was removed ...

2017-05-26 Thread dspavlov
GitHub user dspavlov opened a pull request: https://github.com/apache/ignite/pull/2014 ignite-4947: binary basic test suite was removed from code You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-4947

Re: Ignite 2.0 TeamCity Run configurations

2017-05-26 Thread Semyon Boikov
What about 'Ignite H2 Indexing' suite', it does not execute any test? On Fri, May 26, 2017 at 2:18 PM, Dmitry Pavlov wrote: > Hi Semen, > > > > Thank you for this fix. It is correct to replace to IgniteCacheTestSuite4 > in new 2.0 run configurations as we don’t have Binary* analogue. > > > > Ign

Re: Cassandra and Ignite C++ Issue

2017-05-26 Thread Igor Sapego
Cross-posted Igor's answer to SO Best Regards, Igor On Fri, May 26, 2017 at 8:58 AM, Igor Rudyak wrote: > I assume that's because you are using this: > > > > > Current implementation supports only BLOB serialization for binary > objects. We already have a ticket for this: https://issues.apache

Re: Ignite 2.0 TeamCity Run configurations

2017-05-26 Thread Dmitry Pavlov
Hi Semen, Thank you for this fix. It is correct to replace to IgniteCacheTestSuite4 in new 2.0 run configurations as we don’t have Binary* analogue. IgniteBinaryBasicTestSuite was not deleted to avoid failures in old run configurations. I assumed missing test suite will cause failure on teamc

[GitHub] ignite pull request #1968: Ignite 4575 binary enum for test purposes

2017-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/1968 --- 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 ena

[GitHub] ignite pull request #1721: Ignite 4477 Fix IgniteFuture.listen() and IgniteF...

2017-05-26 Thread dkarachentsev
Github user dkarachentsev closed the pull request at: https://github.com/apache/ignite/pull/1721 --- 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

Re: Ignite 2.0 TeamCity Run configurations

2017-05-26 Thread Semyon Boikov
I just found that 'Ignite Cache 4' was not executed on TeamCity since it has from suite name in configuration, I fixed it to use IgniteCacheTestSuite4, is it correct? Aslo why we need both IgniteBinaryObjectsCacheTestSuite3 and IgniteCacheTestSuite3? And why IgniteBinaryBasicTestSuite is not delet

[GitHub] ignite pull request #2013: IGNITE-5259 Minor serialization fix

2017-05-26 Thread dkarachentsev
GitHub user dkarachentsev opened a pull request: https://github.com/apache/ignite/pull/2013 IGNITE-5259 Minor serialization fix (cherry picked from commit b2040b7) You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ig

Re: DiscoverySpi#setDataExchange returns concrete implementation

2017-05-26 Thread Yakov Zhdanov
Hi Nick! Good catch. Please go ahead and submit a patch. --Yakov 2017-05-26 13:09 GMT+03:00 Николай Ижиков : > Hello, guys. > > public interface DiscoverySpi extends IgniteSpi { > > TcpDiscoverySpi setDataExchange(DiscoverySpiDataExchange var1); > > TcpDiscoverySpi setMetricsProvide

DiscoverySpi#setDataExchange returns concrete implementation

2017-05-26 Thread Николай Ижиков
Hello, guys. public interface DiscoverySpi extends IgniteSpi { TcpDiscoverySpi setDataExchange(DiscoverySpiDataExchange var1); TcpDiscoverySpi setMetricsProvider(DiscoveryMetricsProvider var1); ... } Two methods of DiscoverySpi returns concrete implementation. I think we should retu

[jira] [Created] (IGNITE-5297) .NET: DML update via LINQ

2017-05-26 Thread Pavel Tupitsyn (JIRA)
Pavel Tupitsyn created IGNITE-5297: -- Summary: .NET: DML update via LINQ Key: IGNITE-5297 URL: https://issues.apache.org/jira/browse/IGNITE-5297 Project: Ignite Issue Type: New Feature

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

2017-05-26 Thread Pavel Tupitsyn (JIRA)
Pavel Tupitsyn created IGNITE-5298: -- Summary: .NET: DML update via LINQ Key: IGNITE-5298 URL: https://issues.apache.org/jira/browse/IGNITE-5298 Project: Ignite Issue Type: New Feature

[GitHub] ignite pull request #2009: IGNITE-4904 .NET: DML Delete via LINQ

2017-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/2009 --- 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 ena

[GitHub] ignite pull request #2012: IGNITE-4667 - Throw exception on starting client ...

2017-05-26 Thread dkarachentsev
GitHub user dkarachentsev opened a pull request: https://github.com/apache/ignite/pull/2012 IGNITE-4667 - Throw exception on starting client cache when indexed t… …ypes cannot be loaded You can merge this pull request into a Git repository by running: $ git pull https://git

[GitHub] ignite pull request #2011: GNITE-5038 BinaryMarshaller might need to use con...

2017-05-26 Thread vldpyatkov
GitHub user vldpyatkov opened a pull request: https://github.com/apache/ignite/pull/2011 GNITE-5038 BinaryMarshaller might need to use context class loader for deserialization IGNITE-5038 BinaryMarshaller might need to use context class loader for deserialization You can merge

Re: Reason for disabling @QuerySqlField and @AffinityKeyMapped annotations on methods

2017-05-26 Thread Vladimir Ozerov
Igor, One more reason why we disallowed both annotations on methods is that binary objects doesn't have "method" concept, it only has fields. As binary mode is the default one it doesn't make sense to allow these annotations on methods any more. On Fri, May 26, 2017 at 8:57 AM, Dmitriy Setrakyan