Re: Spring 6 support for ignite extensions

2024-04-29 Thread Ilya Korol
Hi. In general proposal looks good to me, but I agree with Andrey that sticking to same *major.minor* is too much. From my perspective using same major is enough. It still will make it easier to understand correlation between spring and corresponding extension module, meanwhile preserving enough

Re: [DISCUSSION] IEP-107 Logging in Ignite 3

2023-06-15 Thread Ilya Korol
Hi Konstantin, Btw, Ignite 2 has also public methods that accepts a Marker.     public void info(String marker, String msg); Was this part of previous Implementation considered as redundant? I'm concerned because, despite it might not be needed for Ignite 3 itself, IgniteLogger is also a

Re: [DISCUSSION] IEP-107 Logging in Ignite 3

2023-06-08 Thread Ilya Korol
Hi Konstantin. Thanks for you work. Despite you proposal is more about public API of logging Ignite facility, I believe that this thread is the best place to highlight some topic about internal details of IgniteLogger based on my experience with Ignite 2 I already tried to initiate a

Re: IgniteLogger improvement

2022-11-09 Thread Ilya Korol
, Aleksandr Polovtsev пишет: Thank you for the proposal, Ilya! Looks like a really nice change. I can't see any of the mentioned attachments for some reason, though On Sun, Nov 6, 2022 at 10:05 AM Ilya Korol wrote: Hi Igniters, I would like to discuss the Ignite logging. I found current logging

IgniteLogger improvement

2022-11-06 Thread Ilya Korol
Hi Igniters, I would like to discuss the Ignite logging. I found current logging implementation a little bit inflexible for several use-cases. Assume that I want to run multiple Ignite instances in same JVM process and I also want that all messages from each Ignite instance go to separate

Re: [VOTE] Add micronaut dependency to Ignite 3 (reopened)

2022-06-01 Thread Ilya Korol
+1 01.06.2022 19:14, ткаленко кирилл пишет: + 1

Re: [DISCUSSION] IEP-88: CLI Tool

2022-05-19 Thread Ilya Korol
From my perspective we should move towards existing UX approaches. For example: python - enters shell python --help  - prints help python -c - executes command What about other CLI tools that works with remote services? I can't remember properly but do AWS, openshift or maybe GCP CLIs also

IndexOutOfBoundsException in GridCacheWriteBehindStore Flusher thread lookup (IGNITE-14893)

2021-06-22 Thread Ilya Korol
Hi, All. I'm going to fix issue with IndexOutOfBoundsException in GridCacheWriteBehindStore#flusher(K key) method. https://issues.apache.org/jira/browse/IGNITE-14893 This could happen when flush thread count is not a power of 2 an we fallback to mapping via modulo operation: idx = ((h =

IndexOutOfBoundsException in GridCacheWriteBehindStore Flusher thread lookup (IGNITE-14893)

2021-06-15 Thread Ilya Korol
Hi, All. I'm going to fix issue with IndexOutOfBoundsException in GridCacheWriteBehindStore#flusher(K key) method. https://issues.apache.org/jira/browse/IGNITE-14893 This could happen when flush thread count is not a power of 2 an we fallback to mapping via modulo operation: idx = ((h =

Re: Default Lucene index for String cache values

2021-06-03 Thread Ilya Korol
uceneIndex, as there is no table with String as value class. You can check it with my code just replace Long and String classes in cache definition. [1] https://gist.github.com/timoninmaxim/e477ddfcbe56ec9892c7ba6ad44bfadb On Thu, Jun 3, 2021 at 10:56 AM Ilya Korol wrote: Thanks for feedback Maksim,

Re: Default Lucene index for String cache values

2021-06-03 Thread Ilya Korol
just create a simple cache (without entities, indexed types) with String.class as value it won't be indexed, as indexes created per table, not per cache. Do I miss something? On Wed, Jun 2, 2021 at 12:56 PM Ilya Korol wrote: Hi, All. According to https://issues.apache.org/jira/browse/IGNITE-1

Default Lucene index for String cache values

2021-06-02 Thread Ilya Korol
Hi, All. According to https://issues.apache.org/jira/browse/IGNITE-14805 there is default index creation for caches with String values: if (type().valueClass() == String.class) {     try {     luceneIdx = new GridLuceneIndex(idx.kernalContext(), tbl.cacheName(), type);     }     catch

Re: Building with maven 3.8.1

2021-05-24 Thread Ilya Korol
on formatting issue. Otherwise looks good! Considering 3.8.1 maven support — we will be migrating builds there after TC 2021.1 will be delivered. On 20 May 2021, at 19:22, Ilya Korol wrote: Hi, all. Maybe someone has already faced the issue with Ignite and latest Maven release 3.8.1? https

Building with maven 3.8.1

2021-05-20 Thread Ilya Korol
Hi, all. Maybe someone has already faced the issue with Ignite and latest Maven release 3.8.1? https://issues.apache.org/jira/browse/IGNITE-14753 From 3.8.1 maven supplied with config that will block any http repository/mirror. (See details here

Re: Proposal to remove explicit "GC disable" startup suggestion

2021-05-16 Thread Ilya Korol
, Out of curiosity - what are the potential problems with memory buffers that you're referring to? -Val On Fri, May 14, 2021 at 5:06 AM Ilya Korol wrote: Hi, everyone. There is a proposal to remove suggestion that user should disable explicit GC calls in https://issues.apache.org/jira/browse

Proposal to remove explicit "GC disable" startup suggestion

2021-05-14 Thread Ilya Korol
Hi, everyone. There is a proposal to remove suggestion that user should disable explicit GC calls in https://issues.apache.org/jira/browse/IGNITE-14720. Nowadays people usually don't use this facility directly without huge need (at least we hope so), so this suggestion doesn't bring much

Greetings

2021-05-12 Thread Ilya Korol
Hello Ignite Community! My name isIlya Korol. I want to contribute to Apache Ignite. My JIRA username*Korol*. Thanks!