Re: IGNITE-3066 Set of Redis commands that can be easily implemented via existing REST commands

2016-11-18 Thread Valentin Kulichenko
Hi Roman, Do you mean that key expiration is not supported in Ignite? If so, then this is not true, see the last example here: https://apacheignite.readme.io/docs/expiry-policies -Val On Fri, Nov 18, 2016 at 7:55 PM, Roman Shtykh wrote: > Dmitriy, > I think additions of some operations on sets

Re: IGNITE-3066 Set of Redis commands that can be easily implemented via existing REST commands

2016-11-18 Thread Roman Shtykh
Dmitriy, I think additions of some operations on sets and hashes are pretty straightforward. But I think we need to extend REST implementation to cover even more Redis commands (it won't probably remove all limitations we have now though, like not having expiration on keys), or create a Redis pr

Re: Geospatial queries example fails in master

2016-11-18 Thread Denis Magda
Well, The example still fails in the multi-node mode (SpatialQueryExampleMultiNodeSelfTest) which means that a similar filtering, that is applied in the patch from the previous message, has to be done in some other place. Unfortunately, I can’t keep at investigating the issue at the moment any

Re: IgniteCache.loadCache improvement proposal

2016-11-18 Thread Valentin Kulichenko
Alexandr, This has been tested many times already by our users and the answer is simple - it depends :) Any approach has its pros and cons and you never know which one will better for particular use case, database, data model, hardware, etc. Having said that, you will never find the best way to l

[jira] [Created] (IGNITE-4250) Add Ignition.getOrStart method that starts from Spring XML file

2016-11-18 Thread Valentin Kulichenko (JIRA)
Valentin Kulichenko created IGNITE-4250: --- Summary: Add Ignition.getOrStart method that starts from Spring XML file Key: IGNITE-4250 URL: https://issues.apache.org/jira/browse/IGNITE-4250 Project

Re: Geospatial queries example fails in master

2016-11-18 Thread Denis Magda
Andrey,Ok, now I see why the functionality has been broken. Basically, to fix the issue we should avoid the conversion of all “com.vividsolutions.jts” parameters to the binary form. Please see in attach a possible fix. It would be more elegant to register the geospatial classes in BinaryContext but

[jira] [Created] (IGNITE-4249) ODBC: Even simple request takes a lot of time.

2016-11-18 Thread Igor Sapego (JIRA)
Igor Sapego created IGNITE-4249: --- Summary: ODBC: Even simple request takes a lot of time. Key: IGNITE-4249 URL: https://issues.apache.org/jira/browse/IGNITE-4249 Project: Ignite Issue Type: Bug

[jira] [Created] (IGNITE-4248) Slf4jLogger ignores IGNITE_QUIET system property

2016-11-18 Thread Valentin Kulichenko (JIRA)
Valentin Kulichenko created IGNITE-4248: --- Summary: Slf4jLogger ignores IGNITE_QUIET system property Key: IGNITE-4248 URL: https://issues.apache.org/jira/browse/IGNITE-4248 Project: Ignite

Re: Geospatial queries example fails in master

2016-11-18 Thread Andrey Mashenkov
Query parameters are converted to binary since commit "ae77653". This is result of IGNITE-2208[1]. However, spatial queries work fine with OptimizedMarshaller configured. We already have test GridH2IndexingGeoSelfTest, and we can run it with BinaryMarshaller as well. [1] https://issues.apache.or

[jira] [Created] (IGNITE-4247) Table aliases not supported for SqlQuery

2016-11-18 Thread Valentin Kulichenko (JIRA)
Valentin Kulichenko created IGNITE-4247: --- Summary: Table aliases not supported for SqlQuery Key: IGNITE-4247 URL: https://issues.apache.org/jira/browse/IGNITE-4247 Project: Ignite Issue

ignite-spark module in Hadoop Accelerator

2016-11-18 Thread Valentin Kulichenko
Folks, Is there anyone who understands the purpose of including ignite-spark module in the Hadoop Accelerator build? I can't figure out a use case for which it's needed. In case we actually need it there, there is an issue then. We actually have two ignite-spark modules, for 2.10 and 2.11. In Fab

[GitHub] ignite pull request #1249: IGNITE-4246 .NET: Fix IgniteDbConfiguration initi...

2016-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/1249 --- 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

Re: Geospatial queries example fails in master

2016-11-18 Thread Denis Magda
2. Most likely we don’t need to convert the spatial objects into the binary form. I would investigate what has changed since the recent release where everything worked fine. Also don’t forget to add a test for the broken functionality. — Denis > On Nov 18, 2016, at 8:54 AM, Sergi Vladykin wro

Re: Geospatial queries example fails in master

2016-11-18 Thread Sergi Vladykin
1. The change itself is buggy because if the parameter is null, then we will call setNull and setObject after that. It must be `else if ...` 2. I'm not sure how this deserialization fits to our "no classes on server" ideology. Sergi 2016-11-18 14:26 GMT+03:00 Andrey Mashenkov : > Hi Denis, >

[GitHub] ignite pull request #1249: IGNITE-4246 .NET: Fix IgniteDbConfiguration initi...

2016-11-18 Thread ptupitsyn
GitHub user ptupitsyn opened a pull request: https://github.com/apache/ignite/pull/1249 IGNITE-4246 .NET: Fix IgniteDbConfiguration initialization You can merge this pull request into a Git repository by running: $ git pull https://github.com/ptupitsyn/ignite ignite-4246 Alte

Re: IgniteCache.loadCache improvement proposal

2016-11-18 Thread Alexandr Kuramshin
Dmitriy, I will not be fully confident that partition ID is the best approach in all cases. Even if we have full access to the database structure, there are another problems. Assume we have a table PERSON (ID NUMBER, NAME VARCHAR, SURNAME VARCHAR, AGE NUMBER, EMPL_DATE DATE). And we add our colum

[jira] [Created] (IGNITE-4246) .NET: IgniteDbConfiguration initializes incorrectly

2016-11-18 Thread Pavel Tupitsyn (JIRA)
Pavel Tupitsyn created IGNITE-4246: -- Summary: .NET: IgniteDbConfiguration initializes incorrectly Key: IGNITE-4246 URL: https://issues.apache.org/jira/browse/IGNITE-4246 Project: Ignite Issu

[jira] [Created] (IGNITE-4245) Get EXCEPTION_ACCESS_VIOLATION with OFFHEAP_TIRED cache

2016-11-18 Thread Alexandr Kuramshin (JIRA)
Alexandr Kuramshin created IGNITE-4245: -- Summary: Get EXCEPTION_ACCESS_VIOLATION with OFFHEAP_TIRED cache Key: IGNITE-4245 URL: https://issues.apache.org/jira/browse/IGNITE-4245 Project: Ignite

Re: Geospatial queries example fails in master

2016-11-18 Thread Andrey Mashenkov
Hi Denis, Sql query parameters have being converted to binary before query is sent to QueryProcessor. But they haven't being deserialized by QueryProcessor on binding. So, H2 failed to execute query due to wrong parameter type. I've added PR [1] with fix. Sergi, please review if it correct place

Re: IGNITE-3066 Set of Redis commands that can be easily implemented via existing REST commands

2016-11-18 Thread Roman Shtykh
Andrey, I found a problem in INCR/DECR implementation.I think I misunderstood how it works in the current memcached/rest implementation of Ignite, but I modified Redis INCR/DECR now to behave the same way as memcached/rest. Can you please have a look at https://issues.apache.org/jira/browse/IGNIT

[GitHub] ignite pull request #1248: IGNITE-4238: example spatial queries fails

2016-11-18 Thread AMashenkov
GitHub user AMashenkov opened a pull request: https://github.com/apache/ignite/pull/1248 IGNITE-4238: example spatial queries fails Fixed: deserialize binary sql arguments before binding You can merge this pull request into a Git repository by running: $ git pull https://github

[GitHub] ignite pull request #1247: IGNITE-4244: INCR/DECR to operate on AtomicLong.

2016-11-18 Thread shroman
GitHub user shroman opened a pull request: https://github.com/apache/ignite/pull/1247 IGNITE-4244: INCR/DECR to operate on AtomicLong. You can merge this pull request into a Git repository by running: $ git pull https://github.com/shroman/ignite ignite-4244 Alternatively you

[GitHub] ignite pull request #1234: IGNITE-4125 .NET: MultiTieredCacheExample added

2016-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/1234 --- 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 #1245: IGNITE-4129 .NET: Add NearCacheExample

2016-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/1245 --- 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 #1246: IGNITE-4128 .NET: Add EntryProcessorExample

2016-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/1246 --- 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

Re: IGNITE-3066 Set of Redis commands that can be easily implemented via existing REST commands

2016-11-18 Thread Dmitriy Setrakyan
Thanks, Roman! I looked at the content and it looks OK. Are we going to be adding more Redis commands? If yes, would be nice if we would mention which commands we plan to add in the nearest future. D. On Fri, Nov 18, 2016 at 4:23 AM, Roman Shtykh wrote: > Dmitriy, > I have contents prepared fo

[jira] [Created] (IGNITE-4244) Redis INCR/DECR should be able to operate on strings

2016-11-18 Thread Roman Shtykh (JIRA)
Roman Shtykh created IGNITE-4244: Summary: Redis INCR/DECR should be able to operate on strings Key: IGNITE-4244 URL: https://issues.apache.org/jira/browse/IGNITE-4244 Project: Ignite Issue T