Re: [ANNOUNCE] Apache Ignite 3.0.0-beta1 is released

2022-11-21 Thread Valentin Kulichenko
Congrats, folks! This is huge! -Val On Mon, Nov 21, 2022 at 11:16 AM Ivan Daschinsky wrote: > Cool, the first beta is a real achievement! Congrats! > > пн, 21 нояб. 2022 г. в 18:20, Igor Sapego : > > > Congrats, guys! > > > > Best Regards, > > Igor > > > > > > On Thu, Nov 17, 2022 at 4:39 PM

Re: [VOTE] Release Apache Ignite 3.0.0-alpha5 RC1

2022-06-09 Thread Valentin Kulichenko
+1 On Thu, Jun 9, 2022 at 9:17 AM Alexander Polovtcev wrote: > Looks good, so many great features! +1 > > On Thu, Jun 9, 2022 at 6:42 PM Andrey Gura wrote: > > > Dear Community, > > > > In the last few month the following major features have been added: > > > > - Pluggable storages: ability

[ANNOUNCE] Apache Ignite 3.0.0-alpha4 is released!

2022-01-28 Thread Valentin Kulichenko
Igniters, I'm happy to announce that the 4th alpha version of Ignite 3 is out! On top of the functionality that was previously released, Alpha 4 adds the following major features: - Object mappings for table views - DDL - Transactional API and protocol Code examples have been added

[RESULT][VOTE] Release Apache Ignite 3.0.0-alpha4 RC1

2022-01-28 Thread Valentin Kulichenko
Igniters, Apache Ignite 3.0.0-alpha4 RC1 has been accepted. 4 "+1" votes received: - Valentin Kulichenko (binding) - Igor Sapego (binding) - Denis Magda (binding) - Andrey Gura (binding) No "0" or "-1" votes. Vote thread: h

Re: [VOTE] Release Apache Ignite 3.0.0-alpha4 RC1

2022-01-25 Thread Valentin Kulichenko
+1 (binding) On Tue, Jan 25, 2022 at 11:43 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Dear Community, > > Ignite 3 is moving forward and I think we're in a good spot to release > another alpha version. In the last few month the following major features

[VOTE] Release Apache Ignite 3.0.0-alpha4 RC1

2022-01-25 Thread Valentin Kulichenko
Dear Community, Ignite 3 is moving forward and I think we're in a good spot to release another alpha version. In the last few month the following major features have been added: - Transactional API - Record and KeyValue views with POJO mapping support - DDL This is a significant

Re: [VOTE] Service proxy for local service by 'IgniteServices#serviceProxy()'

2022-01-24 Thread Valentin Kulichenko
at 11:53 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > -1 until we finish the discussion in another thread. > > The original problem we're trying to solve here is related to metrics, > which will still be broken because of the service() method. Therefore, in &g

Re: Proxy (GridServiceProxy) for local services if required

2022-01-24 Thread Valentin Kulichenko
r we need extra voting for deprecating > 'service()' and, as earlier and making 'serviceProxy()' return proxy > every time. Several active contributors have said their 'yes' for both > in the thread. > > Several active contributors have already said their 'yes'. > > 23.01.2

Re: Proxy (GridServiceProxy) for local services if required

2022-01-24 Thread Valentin Kulichenko
tand that: we need to revert the previous patch, > update it with the deprecation, and submit it again to master? > > > > On Sun, Jan 23, 2022 at 3:51 AM Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > If we already agreed to deprecate the service() method, then

Re: Proxy (GridServiceProxy) for local services if required

2022-01-22 Thread Valentin Kulichenko
time. > > 2) Deprecate `/service()/`. We've already discussed that here: [1]. > Please see my previous message in the thread. > > > > [1] https://www.mail-archive.com/dev@ignite.apache.org/msg44062.html > > > On 20.01.2022 22:48, Valentin Kulichenko wrote: > >

Re: Incorrect hash and sig links

2022-01-20 Thread Valentin Kulichenko
Hi Mark, Thanks for letting us know. We will fix this. -Val On Thu, Jan 20, 2022 at 8:56 AM Mark Thomas wrote: > Ignite developers, > > I noticed that your download page is incorrectly linking to > dlcdn.apache.org for hashes and signatures. As per the release policy > [1], these links MUST

Re: [VOTE] Service proxy for local service by 'IgniteServices#serviceProxy()'

2022-01-20 Thread Valentin Kulichenko
-1 until we finish the discussion in another thread. The original problem we're trying to solve here is related to metrics, which will still be broken because of the service() method. Therefore, in the way the change is proposed, it doesn't fix the issue, but removes an existing performance

Re: Proxy (GridServiceProxy) for local services if required

2022-01-20 Thread Valentin Kulichenko
e of local/ > > /* services which you can get by, for example, {@link > IgniteServices#service(String)}./ > > > On 20.01.2022 00:20, Valentin Kulichenko wrote: > > BTW, there is also the service() method that can only return an instance > > and never returns a

Re: Proxy (GridServiceProxy) for local services if required

2022-01-19 Thread Valentin Kulichenko
BTW, there is also the service() method that can only return an instance and never returns a proxy. Does it corrupt the metrics as well? -Val On Wed, Jan 19, 2022 at 1:09 PM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Maxim, > > The reason I'm asking is that

Re: Proxy (GridServiceProxy) for local services if required

2022-01-19 Thread Valentin Kulichenko
; > What are the metrics that are being affected by this? > > Only service metrics, that calculates duration of service execution. Check > this ticket [1] > > [1] https://issues.apache.org/jira/browse/IGNITE-12464 > > > On Wed, Jan 19, 2022 at 1:22 AM Valentin Kulichenko &l

Re: Proxy (GridServiceProxy) for local services if required

2022-01-18 Thread Valentin Kulichenko
What are the metrics that are being affected by this? -Val On Tue, Jan 18, 2022 at 3:31 AM Вячеслав Коптилин wrote: > Hello Igniters, > > IMHO, this is not a good idea to change the behavior of serviceProxy() > depending on statistics (enabled/disabled). It seems counterintuitive to > me. >

Ignite 3 Alpha 4

2022-01-16 Thread Valentin Kulichenko
Igniters, Ignite 3 is moving forward and I think we're in a good spot to release another alpha version. In the last few month the following major features have been added: - Transactional API - Record and KeyValue views with POJO mapping support - DDL I've created the release branch based on the

Re: [DISCUSSION] IgniteOutOfMemoryException may not be a critical failure

2022-01-13 Thread Valentin Kulichenko
I tend to agree that providing proper exception to the client is enough in this case, no need to stop server nodes. However, I believe that's how it used to work before we added failure handlers. So probably there was a reason for the current implementation? Does anyone know? -Val On Tue, Jan

Re: [VOTE] @Nullable/@NotNull annotation usage in Ignite 3

2022-01-13 Thread Valentin Kulichenko
+1 for option 2 On Thu, Jan 13, 2022 at 6:17 AM Anton Vinogradov wrote: > +1 to option 4 > > On Thu, Jan 13, 2022 at 5:15 PM Pavel Tupitsyn > wrote: > > > +1 to option 2 > > > > On Thu, Jan 13, 2022 at 3:52 PM Roman Puchkovskiy < > > roman.puchkovs...@gmail.com> wrote: > > > > > +1 to option 2

Re: [VOTE] Release Apache Ignite 2.12.0 RC2

2022-01-10 Thread Valentin Kulichenko
+1 On Mon, Jan 10, 2022 at 7:37 AM Pavel Tupitsyn wrote: > +1 > > Checked .NET binaries, docs, examples, nuget packages. > > On Mon, Jan 10, 2022 at 3:53 PM Nikita Amelchev > wrote: > > > Dear Community, > > > > The release candidate (2.12.0-rc2) is ready. > > > > I have uploaded a release

Re: [DISCUSSION] Shmem removal.

2022-01-07 Thread Valentin Kulichenko
should be removed, I will file a specific > ticket for it. > > ср, 5 янв. 2022 г., 22:26 Valentin Kulichenko < > valentin.kuliche...@gmail.com > >: > > > Hi Ivan, > > > > Do we have a ticket for this? > > > > -Val > > > > On

Re: [DISCUSSION] Shmem removal.

2022-01-05 Thread Valentin Kulichenko
Hi Ivan, Do we have a ticket for this? -Val On Fri, Dec 3, 2021 at 10:58 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > I think we can safely remove it. > > -Val > > On Thu, Dec 2, 2021 at 11:52 PM Ivan Daschinsky > wrote: > >> Hi, igniter

[ANNOUNCE] Welcome Vladislav Pyatkov as a new committer

2021-12-22 Thread Valentin Kulichenko
The Apache Ignite Project Management Committee (PMC) has invited Vladislav Pyatkov to become a new committer and are happy to announce that he has accepted. Vladislav is a veteran of the Apache Ignite project. He joined the community in 2016. He participated in the development of Ignite 1.x, 2.x

Re: [DISCUSSION] @Nullable/@NotNull annotation usage in Ignite 3

2021-12-21 Thread Valentin Kulichenko
be avoided as much as > possible, but not all developers share this point of view. And > especially in Ignite codebase it was quite common to use nullable > variables, for the first time it was very unusual for me. > > 2021-12-20 22:06 GMT+03:00, Valentin Kulichenko < > valentin.ku

Re: [DISCUSSION] @Nullable/@NotNull annotation usage in Ignite 3

2021-12-20 Thread Valentin Kulichenko
Neither solution is completely bulletproof, and I don't think that's what we are looking for. We need something that provides a reasonable value, but also does not clutter the code with too many annotations. I would also keep in mind that annotations are used not only for static analysis, but by

Re: [VOTE] Release Apache Ignite 2.11.1 RC2

2021-12-20 Thread Valentin Kulichenko
+1 On Mon, Dec 20, 2021 at 5:39 AM Alex Plehanov wrote: > +1 > > Checked build from the source, cluster startup with 2 nodes. > > пн, 20 дек. 2021 г. в 16:27, Nikolay Izhikov : > > > +1 (binding) > > > > > 20 дек. 2021 г., в 16:20, Ivan Daschinsky > > написал(а): > > > > > > +1 from me > > >

Re: [VOTE] Release Apache Ignite 2.11.1 RC1

2021-12-17 Thread Valentin Kulichenko
+1 On Fri, Dec 17, 2021 at 6:01 AM Denis Magda wrote: > +1 (binding) > > - > Denis > > > On Fri, Dec 17, 2021 at 7:20 AM Maxim Muzafarov wrote: > > > The release candidate for the 2.11.1 version is ready. > > > > > > I have uploaded a release candidate to: > >

Re: [DISCUSSION] @Nullable/@NotNull annotation usage in Ignite 3

2021-12-16 Thread Valentin Kulichenko
Same here. The second option seems the most reasonable. -Val On Thu, Dec 16, 2021 at 8:07 AM Alexei Scherbakov < alexey.scherbak...@gmail.com> wrote: > +1 for 2 > > чт, 16 дек. 2021 г. в 18:50, Pavel Tupitsyn : > > > Option 2 seems the most sensible. > > It translates to/from other languages

Re: [DISCUSSION] Remove outdated ignite-sys-cache

2021-12-07 Thread Valentin Kulichenko
he differences between the system cache and the > > metastorage, and provide a transition guide for plugin developers. > > > > > > Don’t think we need any transition guide. > > > > > > > I don't think it's reasonable to do this earlier than mid

Re: [DISCUSSION] Remove outdated ignite-sys-cache

2021-12-06 Thread Valentin Kulichenko
t; >> > > >>> Folks, can anyone explain the rush? > > >> > > >> No rush from my side. > > >> > > >> Just want to understand your vision of integration points between > Ignite > > >> and plugins. > > >> &g

Re: [DISCUSSION] Remove outdated ignite-sys-cache

2021-12-03 Thread Valentin Kulichenko
en their app breaks suddenly because of a library > >> update. > >>>>> > >>>>> We know about one use case for sys-cache in GridGain, but there may > be > >>>>> more, no one knows. > >>>>> Every breaking change should be c

Re: [DISCUSSION] Shmem removal.

2021-12-03 Thread Valentin Kulichenko
I think we can safely remove it. -Val On Thu, Dec 2, 2021 at 11:52 PM Ivan Daschinsky wrote: > Hi, igniters. > > Recently I've discovered one fact > 1. GridShmemCommunicationClient and all shmem functionality are broken > since 2.10. In master it is broken since August 2020. And nobody have >

Re: [DISCUSSION] Remove outdated ignite-sys-cache

2021-12-01 Thread Valentin Kulichenko
ard solution plugins can create their own internal > > caches and use them ever they like (or use the metastorage as I > > mentioned earlier). Moving system cache to plugin doesn't look so > > complicated and harmful; > > > > On Mon, 29 Nov 2021 at 23:07, Valentin

Re: [DISCUSSION] Remove outdated ignite-sys-cache

2021-11-29 Thread Valentin Kulichenko
Maxim, You're right that the system cache is still likely to be used by plugins. We at GridGain use it for security features, for example. As far as I know, that's not the only case. I also agree that the metastorage should be the preferred way for this kind of purposes, but is there any harm in

Re: IEP-61 Transaction API desing for Ignite 3

2021-11-29 Thread Valentin Kulichenko
cases as well. What do you think? -Val On Mon, Nov 29, 2021 at 10:59 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > I like Alexei's suggestion. This seems to be the most transparent and > explicit approach. Basically, this ensures that the user is always aware

Re: IEP-61 Transaction API desing for Ignite 3

2021-11-29 Thread Valentin Kulichenko
I like Alexei's suggestion. This seems to be the most transparent and explicit approach. Basically, this ensures that the user is always aware of whether an operation is enlisted in a transaction or not. Any other option is either error-prone, or introduces unnecessary counter-intuitive

[ANNOUNCE] Apache Ignite 3.0.0-alpha3 is released!

2021-10-21 Thread Valentin Kulichenko
Igniters, I'm happy to announce that the third alpha version of Ignite 3 is out! On top of the functionality that was previously released, Alpha 3 adds the following major features: - New SQL engine based on Apache Calcite with implemented JDBC driver. - Persistence implementation based

Re: [DISCUSS] Custom service proxy context

2021-10-19 Thread Valentin Kulichenko
ns for them. > > > Unfortunately, we are currently unable to do the same in .Net because > > > such a feature is not supported in C# 4.0 (it's available in C# 8.0). > > > > > > Can we add default "no-op" implementations for init/execute/cancel >

Re: [DISCUSS] Custom service proxy context

2021-10-19 Thread Valentin Kulichenko
; only > > >>>>> worried about exposing this to the end user. > > >>> > > >>> I'm talking about not Ignite auth, but external auth. Here I am > > >> considering > > >>> Ignite Service Grid as a microservice platform. > &

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-10-16 Thread Valentin Kulichenko
sure that users will not use a wrong JIRA project often? > > 2021-10-05 2:50 GMT+03:00, Valentin Kulichenko < > valentin.kuliche...@gmail.com>: > > Ivan, > > > > I'm not pushing, I'm trying to apply the lazy consensus. It soon will be > a > > whole

[CANCEL][VOTE] Create separate Jira project and Confluence space for Ignite 3

2021-10-16 Thread Valentin Kulichenko
Cancelling the vote due to controversial opinions. We will look for alternative solutions. -Val

Re: [VOTE] Create separate Jira project and Confluence space for Ignite 3

2021-10-16 Thread Valentin Kulichenko
gt; from scratch (from the code point of view). > > > > > > > > ср, 6 окт. 2021 г. в 00:23, Maxim Muzafarov : > > > > > > > > > - 0 from me. > > > > > > > > > > This is OK if the projects are different,

[RESULT][VOTE] Release Apache Ignite 3.0.0-alpha3 RC1

2021-10-16 Thread Valentin Kulichenko
Igniters, Apache Ignite 3.0.0-alpha3 RC1 has been accepted. 3 "+1" votes received: - Denis Magda (binding) - Saikat Maitra (binding) - Pavel Tupitsyn (binding) No "0" or "-1" votes. Vote thread:

Re: [VOTE] Release Apache Ignite 3.0.0-alpha3 RC1

2021-10-16 Thread Valentin Kulichenko
The vote is successful with three binding +1 votes. -Val On Fri, Oct 15, 2021 at 12:21 AM Pavel Tupitsyn wrote: > Val, > > Good point, let's upload nupkg files to SVN for both 2.x and 3.x RCs. > > On Thu, Oct 14, 2021 at 11:49 PM Valentin Kulichenko < > valentin.kulich

Re: [VOTE] Release Apache Ignite 3.0.0-alpha3 RC1

2021-10-14 Thread Valentin Kulichenko
at "Note that NuGet, unfortunately, > > has > > > >> no > > > >> concept of "staging" (unlike Maven). A package with the given > version > > > can > > > >> be published only once, and it can't be undone. We can only p

Re: [VOTE] Release Apache Ignite 3.0.0-alpha3 RC1

2021-10-14 Thread Valentin Kulichenko
know what we are going to upload and can check that everything is > > right. > > > > WDYT? > > > > Best Regards, > > Igor > > > > > > On Wed, Oct 13, 2021 at 8:03 PM Valentin Kulichenko < > > valentin.kuliche...@gmail.com> wrote: > > &

Re: [VOTE] Release Apache Ignite 3.0.0-alpha3 RC1

2021-10-13 Thread Valentin Kulichenko
at, will you be okay if we proceed with the release, and upload the NuGet package after the vote is accepted? We can then have a separate discussion on the overall packaging approach. -Val On Wed, Oct 13, 2021 at 9:57 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: >

Re: [VOTE] Release Apache Ignite 3.0.0-alpha3 RC1

2021-10-13 Thread Valentin Kulichenko
gt; > >> +1 (binding) > >> > >> Regards > >> Saikat > >> > >> On Tue, Oct 12, 2021 at 5:03 PM Denis Magda wrote: > >> > >> > +1 (binding) > >> > > >> > - > >> > Denis > >> > &

[VOTE] Release Apache Ignite 3.0.0-alpha3 RC1

2021-10-12 Thread Valentin Kulichenko
Dear Community, Ignite 3 development is moving forward. In the last several months we've added the following features: - New SQL engine based on Apache Calcite + JDBC driver. - Persistence implementation based on RocksDB. - New binary client protocol with an implementation in Java. -

Re: [DISCUSS] Custom service proxy context

2021-10-11 Thread Valentin Kulichenko
ders", it can > > drastically improves this aspects of our service grid framework. > > > > > > пн, 11 окт. 2021 г. в 00:48, Valentin Kulichenko < > > valentin.kuliche...@gmail.com>: > > > >> I agree with Pavel. The suggested approach is i

Re: [DISCUSS] Custom service proxy context

2021-10-10 Thread Valentin Kulichenko
I agree with Pavel. The suggested approach is indeed utilized quite frequently, but it's inherently error-prone. The main issue is that it creates implicit assumptions about the behavior of both the service and the user's code. For example, if the user's code must provide a username, what if it

Re: [VOTE] Create separate Jira project and Confluence space for Ignite 3

2021-10-04 Thread Valentin Kulichenko
+1 On Mon, Oct 4, 2021 at 4:52 PM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Hello Community, > > As discussed in [1], I would like to propose the creation of a separate > Jira project and Confluence space for Ignite 3. > > Ignite 2 and Ignite 3 a

[VOTE] Create separate Jira project and Confluence space for Ignite 3

2021-10-04 Thread Valentin Kulichenko
Hello Community, As discussed in [1], I would like to propose the creation of a separate Jira project and Confluence space for Ignite 3. Ignite 2 and Ignite 3 are developed in parallel in separate repos, so we need a clear separation in other tools as well - this will help to streamline the

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-10-04 Thread Valentin Kulichenko
t; >>>> > >>>> Respect to the developers who have courage to develop such complex > >>>> things from scratch. > >>>> > >>>>> 29 сент. 2021 г., в 12:55, Petr Ivanov > >>>>> нап

Re: Model of permissions for Ignite 3

2021-09-28 Thread Valentin Kulichenko
Hi Denis, Did you make any progress on this topic? -Val On Wed, Apr 21, 2021 at 6:11 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Danis, > > Got it, thanks. Please provide the link to the IEP when you have one, I’d > be happy to review! > > -Val

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-09-28 Thread Valentin Kulichenko
. If there is no clear picture on option #2 by then, I suggest we go with #1. -Val On Tue, Sep 28, 2021 at 11:22 PM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Folks, > > Versioning is a separate topic. We agreed on the current scheme in March > [1]. If someone thinks

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-09-28 Thread Valentin Kulichenko
;>>> > >>>>> I like the major version update like Ignite 3.0 but if we were to > come > >>>>> up > >>>>> with a name my other suggestion would be > >>>>> > >>>>> Ignite-kernel > >>>>>

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-09-25 Thread Valentin Kulichenko
y released > versions 3. A different GitHub project is not that disturbing. > > - > Denis > > > On Fri, Sep 24, 2021 at 4:09 AM Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > Denis, > > > > From a purely technical perspective

Re: Support for Apache Spark 3.1.2

2021-09-23 Thread Valentin Kulichenko
Hello Abhinav, Spark 3 support definitely becomes higher priority for Ignite as more people transition to it from Spark 2. I'm sure someone in the community will pick this up soon. In the meantime, could you please give a little more detail on how you use Ignite with Spark? Are you using the

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-09-21 Thread Valentin Kulichenko
oo urgent. > > Sincerely, > Dmitriy Pavlov > > On 2021/09/21 10:37:42, Valentin Kulichenko > wrote: > > Hi Dmitry, > > > > According to Infra, this has to be done through > http://selfserve.apache.org/, > > but only PMC chairs have access. > > >

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-09-21 Thread Valentin Kulichenko
Hi Dmitry, According to Infra, this has to be done through http://selfserve.apache.org/, but only PMC chairs have access. Could you please assist with the creation of the Jira project and Confluence space? -Val On Tue, Sep 21, 2021 at 10:46 AM Valentin Kulichenko < valentin.kuli

Re: [DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-09-21 Thread Valentin Kulichenko
and it is not clear where to put them at the moment. > > > > On Sat, Sep 18, 2021 at 4:22 AM Valentin Kulichenko < > > valentin.kuliche...@gmail.com> wrote: > > > >> Igniters, > >> > >> I think it's clear to all of us that Ignite 2.

[DISCUSSION] Separate Jira project and Confluence space for Ignite 3

2021-09-17 Thread Valentin Kulichenko
Igniters, I think it's clear to all of us that Ignite 2.x and 3.x will coexist for a while. They are developed in separate Git repos, but we still accumulate the tickets for both versions in the same Jira project, which seems to complicate the ticket management. For example, we use the

Re: Replace Map with List and Iterable in KeyValueView Ignite 3 APIs

2021-09-09 Thread Valentin Kulichenko
How do we handle the "equality" part in 2.x? The same problem exists there as well, but we still somehow return a Map. Generally, I like Pavel's ideas, but there are a couple of issues with them. Firstly, Java developers are really used to maps in this context. Introducing something else might be

Re: Ban Java Streams usage in Ignite 3 code

2021-09-08 Thread Valentin Kulichenko
lers [1]. > > [1] https://github.com/dotnet/roslyn/blob/main/CONTRIBUTING.md > > On Wed, Sep 8, 2021 at 9:49 PM Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > I don't think we should ban anything. Streams API is just a tool in the > >

Re: Ban Java Streams usage in Ignite 3 code

2021-09-08 Thread Valentin Kulichenko
I don't think we should ban anything. Streams API is just a tool in the toolbox - it should be used appropriately. It's up to the contributor and reviewer(s) to identify whether a particular usage might cause performance issues. -Val On Wed, Sep 8, 2021 at 8:01 AM Alexander Polovtcev wrote: >

Re: Sync vs async APIs in Ignite 3

2021-09-08 Thread Valentin Kulichenko
ve a lot of boiler plate/wrapper that > we > > wrote to get what you're suggesting here. > > > > Regards, > > Courtney Robinson > > Founder and CEO, Hypi > > Tel: ++44 208 123 2413 (GMT+0) <https://hypi.io> > > > > <https://hypi.io> &g

Re: IEP-78 .NET Thin Client for Ignite 3.0

2021-09-07 Thread Valentin Kulichenko
Hi Pavel, I've created a thread. -Val On Mon, Sep 6, 2021 at 12:02 PM Pavel Tupitsyn wrote: > Val, > > Would you like me to start the discussion about sync-over-async in Ignite 3 > Java APIs, or do you plan to do it yourself? > > On Fri, Sep 3, 2021 at 10:10 PM V

Sync vs async APIs in Ignite 3

2021-09-07 Thread Valentin Kulichenko
Igniters, I would like to gather some opinions on whether we want to focus on sync vs async APIs in Ignite 3. Here are some initial considerations that I have: 1. Ignite 2.x is essentially "sync first". Async APIs exist, but they use non-standard IgniteFuture and provide counterintuitive

Re: IEP-78 .NET Thin Client for Ignite 3.0

2021-09-03 Thread Valentin Kulichenko
n affect performance > > > However, I'm not so sure about Java, where async/await are not present, > overall async usage seems to be rarer, and removing sync methods may become > an obstacle for the users in some cases. > Let's create a separate discussion and see what others thin

Re: IEP-78 .NET Thin Client for Ignite 3.0

2021-09-02 Thread Valentin Kulichenko
Hi Pavel, I've looked at the IEP and the public API - looks good to me. Quick question - do you plan to add sync methods to the interfaces, or you're thinking to only leave async? If the latter, what are the arguments for this? The reason I'm asking is that I'm actually thinking about suggesting

Re: [DISCUSSION] Code style for Ignite 3

2021-08-25 Thread Valentin Kulichenko
_checks.xml > > > > On Fri, Aug 20, 2021 at 12:02 PM Alexei Scherbakov > > wrote: > > > > > > +1 > > > > > > пт, 20 авг. 2021 г. в 10:54, Alexander Polovtcev < > > alexpolovt...@gmail.com>: > > > > > > > Hi,

Re: Ignite 3 Java Thin Client configuration: HOCON and ignite-configuration module

2021-08-21 Thread Valentin Kulichenko
Pavel, The configuration framework is designed to support dynamic configuration changes - I doubt this is needed for the client side. I think we should start with simple POJOs or builders. -Val On Sat, Aug 21, 2021 at 7:03 AM Ivan Daschinsky wrote: > As for me, it is very strange purpose and

[DISCUSSION] Code style for Ignite 3

2021-08-19 Thread Valentin Kulichenko
Igniters, I would like to discuss a potential change to the coding guidelines for Ignite 3. Currently, we're using the existing guidelines inherited from Ignite 2, which are described here: https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines Current guidelines, however, exist

Re: Apache Ignite 3 Alpha 2 webinar follow up questions

2021-07-26 Thread Valentin Kulichenko
Hi Courtney, Generally speaking, query caching certainly makes sense. As far as I know, Ignite 2.x actually does that, but most likely there might be room for improvement as well. We will look into this. As for the SQL API - the answer is yes. The requirement for a dummy cache is an artifact of

Re: Text Queries Support

2021-07-23 Thread Valentin Kulichenko
Atri, Sure, go ahead. Let's put the ideas on paper and have a discussion. -Val On Fri, Jul 23, 2021 at 10:59 AM Atri Sharma wrote: > Thanks Andrey. > > I have collected answers or proposals to many of these questions and > would like to start a wiki page covering what we can do for Ignite 3.

Re: Text Queries Support

2021-07-23 Thread Valentin Kulichenko
: > The standard ways to deal with text based searches in SQL are the > CONTAINS operator, the LIKE operator or specific functions > (REGEXP_MATCHES, for eg). I do not think any of these are supported by > Calcite at the moment. > > On Fri, Jul 23, 2021 at 11:20 PM Valentin Kulichenko >

Re: Text Queries Support

2021-07-23 Thread Valentin Kulichenko
In my experience, one of the biggest usability issues with the current support of text queries is that they are completely decoupled from SQL. I.e. you can either execute a SQL query OR a text query. Modern databases, on the other hand, typically allow creating text-based indexes within regular

Re: Ignite 3.0 Ignition API, node startup, and thin client startup

2021-07-12 Thread Valentin Kulichenko
Pavel Tupitsyn wrote: > Val, > > My suggestion is to have Ignition class in ignite-client module. > > On Fri, Jul 9, 2021 at 11:01 PM Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > Pavel, > > > > Ivan actually brings a good point.

Re: Ignite 3.0 Ignition API, node startup, and thin client startup

2021-07-09 Thread Valentin Kulichenko
ally) have both sync and async variants > of > > > > every > > > > > > > method, where applicable, > > > > > > > including the method that connects the client to the cluster. > > > > > > > > > > > > > >

Re: Ignite 3.0 Ignition API, node startup, and thin client startup

2021-07-08 Thread Valentin Kulichenko
.baeldung.com/java-redis-lettuce > > чт, 8 июл. 2021 г., 23:47 Valentin Kulichenko < > valentin.kuliche...@gmail.com > >: > > > Ivan, > > > > Can you please clarify what you mean by "separate creation of client and > > connection"? Can you give a

Re: Ignite 3.0 Ignition API, node startup, and thin client startup

2021-07-08 Thread Valentin Kulichenko
I > > - Make Ignition a class, not an interface > > - Add static Ignition#startClient > > > > Sounds good? > > > > On Thu, Jul 8, 2021 at 6:13 AM Valentin Kulichenko < > > valentin.kuliche...@gmail.com> wrote: > > > > > Hi Iv

Re: Ignite 3.0 Ignition API, node startup, and thin client startup

2021-07-08 Thread Valentin Kulichenko
t; On Thu, Jul 8, 2021 at 6:13 AM Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > Hi Ivan, > > > > Ignition IS the entry point to Ignite, so I'm not sure I got your point > :) > > Where is the contradiction? > > > > Either way, p

Re: Ignite 3.0 Ignition API, node startup, and thin client startup

2021-07-07 Thread Valentin Kulichenko
lly “Ignition” naming always confused me. I think about > it as some fancy named API entry point for Ignite. Perhaps it is a good > moment to revisit naming. > > > On 8 Jul 2021, at 07:09, Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > > Hi Pa

Re: Ignite 3.0 Ignition API, node startup, and thin client startup

2021-07-07 Thread Valentin Kulichenko
Hi Pavel, I don't think we will need the pure embedded mode, but we still need to be able to access the API from compute and services. That said, there are two usages of the 'Ignite' API: 1. Remote, via the binary protocol. 2. Local - needed for compute and services. (This is how it works

Re: Ignite 3.0 Tuple API: how to check if value is null?

2021-07-06 Thread Valentin Kulichenko
orner.com/article/out-parameter-in-c-sharp-7/ > > > вт, 6 июл. 2021 г., 22:30 Valentin Kulichenko < > valentin.kuliche...@gmail.com > >: > > > Pavel, > > > > Optionals are available in Java and we can use them. This is still boxing > > though, and I don't k

Re: IEP-76 Thin Client Protocol for Ignite 3.0

2021-07-06 Thread Valentin Kulichenko
>> > > >> As for me, I want to take part in implementing python and golang thin > > >> clients for ignite 3, so consider my remarks using this info. I am not > > >> just > > >> a rude critic, > > >> I am just interested in con

Re: Ignite 3.0 Tuple API: how to check if value is null?

2021-07-06 Thread Valentin Kulichenko
> > > > > > Val, > > > > > > > > > I don't think there is a significantly better way > > > > > of doing this in Java. > > > > > > > > Yep looks like there is no way to return two values without boxing. > > > &

Re: Ignite 3.0 Tuple API: how to check if value is null?

2021-07-05 Thread Valentin Kulichenko
Pavel, That's a good point, but I don't think there is a significantly better way of doing this in Java. There should be a way to check if a field is nullable or not though. Schema already provides this information, doesn't it? -Val On Mon, Jul 5, 2021 at 11:03 AM Pavel Tupitsyn wrote: >

Re: IEP-76 Thin Client Protocol for Ignite 3.0

2021-07-01 Thread Valentin Kulichenko
Daschinsky wrote: > Val, am I right, that kv view over the tuples is just simple mapping from > POJO to tuple? No collections, no nested objects? I have discussed this in > private with Igor and Pavel and they told me different info. > > чт, 1 июл. 2021 г., 19:43 Vale

Re: Setting IGNITE_TO_STRING_INCLUDE_SENSITIVE=false prevents VM Arguments output

2021-07-01 Thread Valentin Kulichenko
21 г., 19:51 Valentin Kulichenko < > valentin.kuliche...@gmail.com > >: > > > Ivan, > > > > IP addresses (e.g. IGNITE_TCP_DISCOVERY_ADDRESSES) and file paths > > (e.g. IGNITE_CONFIG_URL) are often considered sensitive information. Data > > related to

Re: Setting IGNITE_TO_STRING_INCLUDE_SENSITIVE=false prevents VM Arguments output

2021-07-01 Thread Valentin Kulichenko
uments even if > sensitive data is restricted? > > What do you think about an extra JVM option? > > чт, 1 июл. 2021 г. в 19:51, Valentin Kulichenko < > valentin.kuliche...@gmail.com>: > > > Ivan, > > > > IP addresses (e.g. IGNITE_TCP_DISCOVERY_ADDRESSES

Re: Setting IGNITE_TO_STRING_INCLUDE_SENSITIVE=false prevents VM Arguments output

2021-07-01 Thread Valentin Kulichenko
efault and block it wheb set to true > > чт, 1 июл. 2021 г., 19:45 Atri Sharma : > > > What if we allowed a blocklist of parameters that are never printed? > > > > On Thu, 1 Jul 2021, 22:06 Valentin Kulichenko, < > > valentin.kuliche...@gmail.com> wrote: >

Re: IEP-76 Thin Client Protocol for Ignite 3.0

2021-07-01 Thread Valentin Kulichenko
Ivan, I was answering your question about the KV API. The API I provided has been discussed and agreed upon. One of the goals of the protocol is to implement this API, so it should give you a clear idea of what we're looking for. Of course, I agree with you that the protocol should be simple and

Re: Setting IGNITE_TO_STRING_INCLUDE_SENSITIVE=false prevents VM Arguments output

2021-07-01 Thread Valentin Kulichenko
ity through obscurity, an obvious and a well-known anti > pattern. I suppose that printing jvm options, that is registered by > @IgniteSystemProperty annotation is an ideal variant > > чт, 1 июл. 2021 г., 19:25 Valentin Kulichenko < > valentin.kuliche...@gmail.com > >: >

Re: Setting IGNITE_TO_STRING_INCLUDE_SENSITIVE=false prevents VM Arguments output

2021-07-01 Thread Valentin Kulichenko
Folks, *Anything* that a user provides to the system can potentially be considered sensitive information. This includes the VM arguments. We can't predict what exactly one can put there, so let's not make assumptions. When dealing with security, we should be as conservative as possible. That

Re: IEP-76 Thin Client Protocol for Ignite 3.0

2021-07-01 Thread Valentin Kulichenko
Ivan, Regarding the API, please take a look at this package: https://github.com/apache/ignite-3/tree/main/modules/api/src/main/java/org/apache/ignite/table 'Table' is the primary API, which works with raw tuples. There are also multiple views on top of it, including KeyValueView and

Re: Ignite 3.0 IgniteTables API Improvement Suggestion

2021-06-30 Thread Valentin Kulichenko
> have a use case in mind for this? If not, I would keep this internal > > > > > > Ok, we can keep the Table.schema(ver) method internal, as long as > > > Table.schemas() is public and includes schema versions. > > > > > > > > > > We already

[ANNOUNCE] Apache Ignite 3.0.0-alpha2 is released!

2021-06-30 Thread Valentin Kulichenko
Igniters, I'm happy to announce that Ignite 3 project reached a significant milestone, as we release the 2nd alpha version of the product. On top of the functionality that was previously released, Alpha 2 adds the following major features: - Replication infrastructure based on Raft. - New

Re: Ignite 3.0 IgniteTables API Improvement Suggestion

2021-06-29 Thread Valentin Kulichenko
Hi Pavel, Please see my comments below. -Val On Tue, Jun 29, 2021 at 2:23 PM Pavel Tupitsyn wrote: > Igniters, > > While working on "IEP-76 Thin Client Protocol for Ignite 3.0" [1] (to be > discussed separately), the following suggestions for the Table API came up: > > 1. Expose table IDs:

  1   2   3   4   5   6   7   8   9   10   >