Re: Re[2]: [DISCUSSION] Apache Ignite Release 2.10 (time, scope, manager)

2021-02-18 Thread Igor Sapego
Maxim, I believe I could fix the ticket [1] by the end of the next week. [1] https://issues.apache.org/jira/browse/IGNITE-14204 Best Regards, Igor On Thu, Feb 18, 2021 at 6:30 PM Max Timonin wrote: > Hi! I've today found an issue [1], there is wrong handling of inlined POJO. > This bug appea

Re: Re[2]: [DISCUSSION] Apache Ignite Release 2.10 (time, scope, manager)

2021-03-01 Thread Igor Sapego
gt; > > > > > Fixed an issue that caused a deadlock when user cache was created > in > > > > > parallel with TTL worker was in progress. > > > > > https://issues.apache.org/jira/browse/IGNITE-14078 > > > > > > > > > > On Thu, 18 Fe

Re: Re[2]: [DISCUSSION] Apache Ignite Release 2.10 (time, scope, manager)

2021-03-01 Thread Igor Sapego
The following commit should be cherry-picked: 0675e2a7e800730c9c8230332b82809754ddae5a Sorry for a delay. Best Regards, Igor On Mon, Mar 1, 2021 at 9:06 PM Igor Sapego wrote: > Maxim, > > The issue is fixed and is merged to master now. > > Best Regards, > Igor > > >

Re: IEP-68: Thin Client Data Streamer

2021-03-05 Thread Igor Sapego
Pavel, I've checked the IEP and I like it. The only thing that seems a bit confusing to me is that there are 4 different variants for clients but there are cons and pros for different variants. Maybe at least few sentences should be written here to give developers who are not familiar with DataStr

Re: [VOTE] Release Apache Ignite 2.10.0 RC1

2021-03-05 Thread Igor Sapego
+1 (binding) Checked C++ compilation, C++ examples Best Regards, Igor On Fri, Mar 5, 2021 at 12:32 AM Denis Magda wrote: > +1 (binding) > > Downloaded the binary package and started a 2-node cluster on MacOS with > ignite.sh. > > - > Denis > > > On Wed, Mar 3, 2021 at 1:03 PM Maxim Muzafarov

Re: Thin client implementation removal

2021-03-15 Thread Igor Sapego
Nikolay, That's because we now have separate repos for them: [1], [2] and [3]. Actually, active development is moved to those repos some time ago and those directories in main repo are not actual anymore anyway. Decision of moving those clients to separate repos was discussed in [4] [1] - https:

Re: IEP-70: Async Continuation Executor

2021-03-16 Thread Igor Sapego
Pavel, I like the proposal, +1 from me Best Regards, Igor On Tue, Mar 16, 2021 at 6:49 PM Pavel Tupitsyn wrote: > Alexey, > > .NET thick API delegates to Java directly. > > When you do ICache.PutAsync(): > * Future is created on Java side, .listen() is called > * TaskCompletionSource is creat

[ANNOUNCE] Welcome Ivan Daschinsky as a new committer

2021-04-12 Thread Igor Sapego
The Project Management Committee (PMC) for Apache Ignite has invited Ivan Daschinsky to become a committer and we are pleased to announce that he has accepted. Ivan made a lot of contributions to Apache Ignite. He helped a lot to improve our Python Thin Client fixing a lot of different bugs and co

Re: [VOTE] Release pyignite 0.4.0-rc1

2021-04-16 Thread Igor Sapego
+1 from me. Best Regards, Igor On Fri, Apr 16, 2021 at 5:05 PM Ivan Daschinsky wrote: > Ivan Daschinsky > чт, 15 апр., 21:37 (19 часов назад) > кому: dev > Dear Igniters! > > Release candidate binaries are at least uploaded and ready for vote > You can find them here: > https://dist.apache.or

Re: [DISCUSSION] Documentation of thin clients (python, php, nodejs)

2021-04-26 Thread Igor Sapego
Hi, Ivan, I like your suggestion. To me it looks better than the current approach. Best Regards, Igor On Mon, Apr 26, 2021 at 11:47 AM Nikita Safonov wrote: > Hi Ivan, > > Thanks for sharing the information. > I'll look through the docs and share my thoughts and suggestions soon. > > Regards,

Re: NodeJS thin client: full API

2021-04-26 Thread Igor Sapego
Thanks for issuing a ticket. I'll take a look at it. Best Regards, Igor On Fri, Apr 23, 2021 at 1:40 PM teligenz.dheeraj wrote: > Team, > > I have used nodejs thin client to connect ignite. With single query at time > on socket works fine. But when hit multiple request simultaneously, getting

Re: Thin Clients: enable partition awareness by default

2021-05-12 Thread Igor Sapego
+1 from me. There were no major issues with this feature and it gives good performance boost for many cases. Best Regards, Igor On Wed, May 12, 2021 at 5:18 PM Ivan Daschinsky wrote: > Huge +1 from me. PA should be enabled by default. > > ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn : > > > > Ig

Re: IEP-68: Thin Client Data Streamer

2021-05-18 Thread Igor Sapego
t this will not work at all. > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > пт, 5 мар. 2021 г. в 15:23, Pavel Tupitsyn < > >> > > ptupit...@apache.org > >> > > > >: > >>

Seconds and milliseconds confusion in python thin client

2021-06-15 Thread Igor Sapego
Hi Igniters, I've noticed a weird behaviour of python thin client. In those places where we have timeouts or any other parameters that take time in some places we treat it like integer number of milliseconds, in others it can take both floats (as a number of seconds) and ints (number of millisecon

Re: [VOTE] Release pyignite 0.5.0-rc0

2021-06-15 Thread Igor Sapego
+1 from me Uploaded to test.pipy.org: https://test.pypi.org/project/pyignite/0.5.0/ Everything looks good. Best Regards, Igor On Tue, Jun 15, 2021 at 10:09 PM Ivan Daschinsky wrote: > Also checked hash sums and signature. Packages are verified and > signature is OK, signed by Igor

Re: Seconds and milliseconds confusion in python thin client

2021-06-15 Thread Igor Sapego
ht. But there is no need to > > notice or deprecate something. This functionality is not released yet > > > > вт, 15 июн. 2021 г., 23:41 Igor Sapego : > > > >> Hi Igniters, > >> > >> I've noticed a weird behaviour of python thin client. In those pla

Re: [VOTE] Release pyignite 0.5.0-rc0

2021-06-16 Thread Igor Sapego
I propose to cancel this release and fix the issue which was highlighted in the "Seconds and milliseconds confusion in python thin client" thread. WDYT? Best Regards, Igor On Wed, Jun 16, 2021 at 12:10 AM Igor Sapego wrote: > +1 from me > > Uploaded to test.pipy.org: ht

Re: [VOTE] Release pyignite 0.5.0-rc1

2021-06-17 Thread Igor Sapego
+1 from me Best Regards, Igor On Thu, Jun 17, 2021 at 12:10 PM Ivan Daschinsky wrote: > +1 From me > Checked on Ubuntu 20.04 and windows 10 > 1. Installation from wheels for pythons 3.6 3.7 3.8 3.9 > 2. Native module work > 3. Examples > > Checked on Ubuntu 20.04 building from source package a

Re: [VOTE] Release pyignite 0.5.0-rc1

2021-06-18 Thread Igor Sapego
.0 so not sure if it’s a regression, but it’s > >>>>> not great. > >>>>> > >>>>> > On 18 Jun 2021, at 09:36, Stephen Darlington < > >>>>> stephen.darling...@gridgain.com> wrote: > >>>>> > > >

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

2021-06-28 Thread Igor Sapego
+1 Best Regards, Igor On Sat, Jun 26, 2021 at 1:41 AM Nikita Ivanov wrote: > +1 > > -- > Nikita Ivanov > > > > On Fri, Jun 25, 2021 at 3:31 PM Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > Dear Community, > > > > In the last several months, the development of Ignite 3 has

Re: IEP-76 Thin Client Protocol for Ignite 3.0

2021-07-01 Thread Igor Sapego
Ivan, what are extra serde steps you are talking about? Best Regards, Igor On Thu, Jul 1, 2021 at 5:52 PM Ivan Daschinsky wrote: > > I agree. But this was decided before in IEP-54, and is out of scope for > current IEP. > Would you like to start a separate thread to discuss this? Or I can do t

Re: [VOTE] Release pyignite 0.5.1-rc0

2021-07-26 Thread Igor Sapego
and gpg signatures (signed by Igor Sapego (CODE > SIGNING KEY) 5C10 A072 2D94 7727 923C 98B5 AF35 DBD9 > 58FE 8DC5) > key is inside https://downloads.apache.org/ignite/KEYS) > > пт, 23 июл. 2021 г. в 13:52, Ivan Daschinsky : > > > The voting finishes at 07/27/2021 12:00 UT

Re: [Announcement] Apache Ignite 2.11 Code Freeze started

2021-07-28 Thread Igor Sapego
Igniters, I suggest adding [1] to the scope of release, because it contains changes to code introduced by [2], which is already included in release. [1] - https://issues.apache.org/jira/browse/IGNITE-14815 [2] - https://issues.apache.org/jira/browse/IGNITE-14658 Best Regards, Igor On Mon, Jul

Re: [Announcement] Apache Ignite 2.11 Code Freeze started

2021-07-29 Thread Igor Sapego
ockers. I may be > wrong, but this ticket doesn't seem to be of that kind. > > On 2021/07/28 21:00:15, Igor Sapego wrote: > > Igniters, > > > > I suggest adding [1] to the scope of release, because it contains > > changes to code introduced by [2], which is a

Re: [Announcement] Apache Ignite 2.11 Code Freeze started

2021-07-29 Thread Igor Sapego
t the incomplete change from 2.11 in order > to reintroduce it in 2.12 in full form if Igor agrees and RE thinks this is > the best course of action. > > Regards, > -- > Ilya Kasnacheev > > > чт, 29 июл. 2021 г. в 18:07, Igor Sapego : > > > Alexey, > > >

Re: [Announcement] Apache Ignite 2.11 Code Freeze started

2021-08-01 Thread Igor Sapego
e it in 2.12 in full form if Igor agrees and RE thinks > this is > > > the best course of action. > > > > > > Regards, > > > -- > > > Ilya Kasnacheev > > > > > > > > > чт, 29 июл. 2021 г. в 18:07, Igor Sapego : > > > > >

Re: Sync vs async APIs in Ignite 3

2021-09-09 Thread Igor Sapego
Well, fortunately we do not provide a C client if you don't consider ODBC as one so we should not think about it. For C++ I believe we should use standard std::future+std::promise for async, but still can provide sync methods, built on top of async methods. There is no continuation problem in C++ A

Re: Sync vs async APIs in Ignite 3

2021-09-09 Thread Igor Sapego
sync API. Best Regards, Igor On Thu, Sep 9, 2021 at 2:29 PM Ivan Daschinsky wrote: > Igor, and what about C++20 and coroutines [1] > > [1] -- https://en.cppreference.com/w/cpp/language/coroutines > > чт, 9 сент. 2021 г. в 14:12, Igor Sapego : > > > Well, fortunately we

Re: Release of pyignite 0.5.2 proposal

2021-09-10 Thread Igor Sapego
+1 Best Regards, Igor On Thu, Sep 9, 2021 at 8:33 PM Maxim Muzafarov wrote: > +1 > > On Thu, 9 Sept 2021 at 14:08, Nikolay Izhikov wrote: > > > > +1 to release ASAP. > > > > > 9 сент. 2021 г., в 13:43, Ivan Daschinsky > написал(а): > > > > > > TC build of release branch -- > > > > https://tc

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

2021-09-10 Thread Igor Sapego
I actually agree with Pavel, at least at putAll() part. We require a Map from user when we do not really need a Map in this method. What we really need here is an iterable collection of pairs. Can not see why user can not pass for example an array here. Now, when we talk about getAll() method it's

Re: Tuple equality in Ignite 3.x

2021-09-10 Thread Igor Sapego
Sounds very reasonable to me. +1 Though the default comparator should be implemented very carefully as we had issues with comparison of binary objects in 2.x Best Regards, Igor On Thu, Sep 9, 2021 at 4:04 PM Pavel Tupitsyn wrote: > Igniters, > > Tuple in Ignite 3.x is a replacement for Binar

Re: API Proposal: Declare IgniteClient::close that throws no exceptions (IGNITE-15688)

2021-10-06 Thread Igor Sapego
Sounds good, no objections from my side. Best Regards, Igor On Wed, Oct 6, 2021 at 11:46 AM Stanislav Lukyanov wrote: > Hi Igniters, > > I found the following usability issue with java thin client API. > > Whenever you do `try (IgniteClient client = Ignition.startClient(cfg))`, > you're forced

Re: [DISCUSS] Custom service proxy context

2021-10-08 Thread Igor Sapego
Hi guys, Why can not a user implement such context on application level? I believe Ignite provides all necessary tools for that. User can just implement such a context as user type and pass it to services they need. Are the arguments why would Ignite need a separate feature for such a use case? B

Re: Updating Javascript npm Package

2021-10-13 Thread Igor Sapego
Kevin, Basically, to change this we need people who would actively drive development of the client and be active community members. Best Regards, Igor On Mon, Sep 27, 2021 at 6:02 PM Ivan Daschinsky wrote: > Hi! I can share my experience how to drive this activity. Personally, I've > driven f

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

2021-10-14 Thread Igor Sapego
Val, I think we need to upload the nuget package we want to upload so the community would 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: > Pavel, > >

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

2021-10-14 Thread Igor Sapego
Sorry, I meant we need to publish the package as part of RC, so it can be reviewed. Best Regards, Igor On Thu, Oct 14, 2021 at 11:34 AM Igor Sapego wrote: > Val, > > I think we need to upload the nuget package we want to upload so the > community > would know what we are goin

Re: IEP-82 Thin Client Retry Policy

2021-11-25 Thread Igor Sapego
Pavel, What is ClientOperationType? Will it list basically all operations or only types like Idempotent, NonIdempotent? Best Regards, Igor On Wed, Nov 24, 2021 at 5:21 PM Pavel Tupitsyn wrote: > Igniters, > > I've prepared a proposal about thin client retry behavior. > Please review and let m

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

2022-01-26 Thread Igor Sapego
+1 (binding) Best Regards, Igor On Tue, Jan 25, 2022 at 10:44 PM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > +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

Re: IEP-83 Thin Client Keepalive (heartbeat)

2022-02-07 Thread Igor Sapego
Feature seems useful for me as it makes connection management more robust and predictable. I agree with Pavel, that we should print warning when heartbeat period is larger than idle timeout, but I see a problem here as idle timeout is configured on server and is not known to clients, while heartbe

IEP-90 Ignite 3 Client Lifecycle

2022-05-16 Thread Igor Sapego
Hi, Igniters I've prepared an IEP for Ignite 3 Client Lifecycle [1]. The main idea is to define client lifecycle as well as core algorithms and mechanisms used by clients. This proposal can be used as a reference for implementation of a new client for Ignite when dealing with such problems as:

Re: IEP-90 Ignite 3 Client Lifecycle

2022-05-17 Thread Igor Sapego
In the case of a secured cluster it does not matter, because > authentication/authorization keeps intruders out. > > > On Mon, May 16, 2022 at 11:07 PM Igor Sapego wrote: > > > Hi, Igniters > > > > I've prepared an IEP for Ignite 3 Client Lifecycle [1]. The main

Re: IEP-90 Ignite 3 Client Lifecycle

2022-05-19 Thread Igor Sapego
>> > >> Also, do I understand correctly that a server has enough information > >> about client connections so it will be possible to observe a > >> connections list on the server? It would be useful for cluster > >> monitoring purposes. > >> > >

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

2022-06-10 Thread Igor Sapego
+1 Best Regards, Igor On Fri, Jun 10, 2022 at 1:37 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > +1 > > On Thu, Jun 9, 2022 at 9:17 AM Alexander Polovtcev < > alexpolovt...@gmail.com> > wrote: > > > Looks good, so many great features! +1 > > > > On Thu, Jun 9, 2022 at 6:42 PM

Re: IgniteSet, thin client, and WeakReferenceCloseableIterator

2022-06-26 Thread Igor Sapego
I like the closable iterator approach more as well. The difference in API is not as critical in my opinion, as we have a lot of differences in thick and thin APIs already and users would normally seek thin client examples and not just re-use thick client code with thin client. Best Regards, Igor

Re: IEP-90 Ignite 3 Client Lifecycle

2022-06-27 Thread Igor Sapego
hu, May 19, 2022 at 10:55 PM Igor Sapego wrote: > > > > Andrey, > > > > 1. If a server generates a UUID that already exists it can check and just > > re-generate it straight away > > as a check is just a simple map lookup. > > > > 2. Well,

Re: [DISCUSSION] IEP-95 Client Partition Awareness (Ignite 3)

2022-09-15 Thread Igor Sapego
Pavel, Great, this feature showed great results in Ignite 2, so It's a good idea to implement it in Ignite 3 as well. The IEP itself looks good to me, except it's not clear how a server would know when assignment has changed. Regardless of the Tracking Assignment Changes section, I like the firs

Re: [ANNOUNCE] SCOPE FREEZE for Apache Ignite 3.0.0 beta 1 RELEASE

2022-11-01 Thread Igor Sapego
Guys, I've merged https://issues.apache.org/jira/browse/IGNITE-17590 to main and I really think it should be included in this release, because without it current C++ client implementation is pretty much useless. And it does not affect any other parts of the product except for the C++ part anyway s

Re: Apache Ignite 3.0.0 beta 1 RELEASE [Time, Scope, Manager]

2022-11-02 Thread Igor Sapego
+1 from me Best Regards, Igor On Wed, Nov 2, 2022 at 3:48 AM Stanislav Lukyanov wrote: > Igniters, > > The initial code freeze date for 3.0.0 beta 1 was missed, so we need to > pick a new timeline. > > There are currently 5 tickets in progress or in review that are in the > scope, with signifi

Re: [ANNOUNCE] Release pyignite-0.6.0

2022-11-08 Thread Igor Sapego
+1 Github Actions look great to me. Best Regards, Igor On Mon, Nov 7, 2022 at 5:38 PM Ivan Daschinsky wrote: > Hi, Igniters! > > I suppose it is time to release pyignite 0.6.0, since we have released our > previous release more than a year ago. > > Firstly, python 3.6 reached its EOL, the bra

Re: [VOTE] Release pyignite 0.6.0.rc0

2022-11-11 Thread Igor Sapego
+1 Best Regards, Igor On Fri, Nov 11, 2022 at 5:41 PM Ivan Daschinsky wrote: > Dear Igniters! > > Release candidate binaries for subj are uploaded and ready for vote > You can find them here: > https://dist.apache.org/repos/dist/dev/ignite/pyignite/0.6.0.rc0 > > If you follow the link above, y

Re: [VOTE] Release pyignite 0.6.0.rc1

2022-11-14 Thread Igor Sapego
+1 Best Regards, Igor On Mon, Nov 14, 2022 at 5:23 PM Ivan Daschinsky wrote: > >> > > https://apache-ignite-binary-protocol-client.readthedocs.io/en/0.6.0.rc0/examples.html > > https://apache-ignite-binary-protocol-client.readthedocs.io/en/0.6.0.rc1/examples.html > > пн, 14 нояб. 2022 г. в 16:

Re: [VOTE] Release Apache Ignite 3.0.0-beta1 RC2

2022-11-16 Thread Igor Sapego
+1 (binding) Best Regards, Igor On Tue, Nov 15, 2022 at 11:10 PM Vladislav Pyatkov wrote: > +1 > > On Tue, Nov 15, 2022 at 3:35 PM Denis C wrote: > > > > +1 > > > > вт, 15 нояб. 2022 г. в 13:33, Alexander Lapin : > > > > > +1 > > > > > > вт, 15 нояб. 2022 г. в 08:48, Pavel Tupitsyn : > > > >

Re: [ANNOUNCE] Apache IGNITE python thin client (pyignite) 0.6.0 have been released

2022-11-17 Thread Igor Sapego
Great work Best Regards, Igor On Wed, Nov 16, 2022 at 1:50 PM Ivan Daschinsky wrote: > The Apache Ignite Community is pleased to announce the release of > Apache IGNITE python thin client (pyignite) 0.6.0. > > This new release is mostly the maintenance one. However, there are some > new import

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

2022-11-21 Thread Igor Sapego
Congrats, guys! Best Regards, Igor On Thu, Nov 17, 2022 at 4:39 PM Вячеслав Коптилин wrote: > Dear Igniters, > > I'm happy to announce that the 1st beta version of Ignite 3 is out! > > On top of the functionality that was previously released, Beta 5 adds the > following major features: > - RPM

Re: Binary object format KB article

2019-10-18 Thread Igor Sapego
Great job, I think we should have details like this in documentation, not only in wiki Denis, what do you think? Best Regards, Igor On Wed, Oct 16, 2019 at 7:19 PM Ivan Pavlukhin wrote: > Sergey, > > Thank you for a review! > > > It seems to me that document tries to focus on details of the

Re: Thin client: compute support

2019-11-21 Thread Igor Sapego
Pavel, The proposed solution won't work for PHP, Python, Node.js, but it will work for C++ and .NET, isn't it? We will just have to deploy C++/.NET code in closter (just as in Java). Best Regards, Igor On Thu, Nov 21, 2019 at 12:59 PM Aleksandr Shapkin wrote: > Folks, > > > > I started to imp

Re: Full-text queries in Thin Client protocol

2019-12-03 Thread Igor Sapego
Pavel, I'm against adding this feature, as there were talks recently, that we should stop supporting TextQuery altogether. No sense in adding something, that we will need to depreciate and remove soon. Best Regards, Igor On Tue, Dec 3, 2019 at 5:46 PM Pavel Tupitsyn wrote: > Oh wow, the next

Re: Thin Clients: Renaming of AffinityAwareness to PartitionAwareness

2019-12-20 Thread Igor Sapego
+1, sounds reasonable to me. Best Regards, Igor On Fri, Dec 20, 2019 at 10:25 AM Pavel Tupitsyn wrote: > +1, let's rename while we have a chance before 2.8 is released. >

Re: Slim binary release and docker image for Apache Ignite

2020-01-16 Thread Igor Sapego
Alexey, if I understand correctly, Ilya does not suggest to pre-built binaries, just to ship it with configure script pre-generated, which is a common practice for autotools packages. Building will be still required for the user, but there will be less requirements and possible errors during build.

Re: Thin client: compute support

2020-01-20 Thread Igor Sapego
Sorry for the late reply. Approach with taskId will require a lot of changes in protocol and thus more "heavy" for implementation, but it definitely looks to me less hacky than reqId-approach. Moreover, as was mentioned, server notifications mechanism will be required in a future anyway with high

Re: Add user attributes to thin clients

2020-01-22 Thread Igor Sapego
Hi Dmitrii, Can you please explain your use case? I'm not sure I'm getting what is the motivation of this change. Best Regards, Igor On Wed, Jan 22, 2020 at 5:11 PM Pavel Tupitsyn wrote: > Hi Dmitrii, > > Honestly, I could not grasp the problem, can you explain it in more detail? > What do we

Feature masks for thin clients

2020-01-23 Thread Igor Sapego
Hi Igniters, As we have a lot of different thin clients now, maintained by different people, the issues with our backward compatibility mechanism becomes more and more prominent. Currently, we use protocol versioning as the only approach to provide backward compatibility. The main issue of this a

Re: Feature masks for thin clients

2020-01-24 Thread Igor Sapego
sion for introducing > flags. > > > > On 23.01.2020 15:47, Alexei Scherbakov wrote: > > > Igor Sapego, > > > > > > I do not understand how feature masks can remove the necessity of > having > > > protocol versioning. > > > A protocol for one featur

Re: Add user attributes to thin clients

2020-01-28 Thread Igor Sapego
gt; > and > > > > > > raises > > > > > > > > > > > > > questions: > > > > > > > > > > > > > > > > > > > > > > > > > > - Why is UserAttributes property related to > > >

Re: Add user attributes to thin clients

2020-01-28 Thread Igor Sapego
; which tries to clear sensitive data from it. > > However, it is no good for authenticating nodes anyway, since discovery > message have to travel via ring, so all server nodes will have access to > sensitive information. > > Regards, > -- > Ilya Kasnacheev > > > в

Re: [ANNOUNCE] New Apache Ignite PMC member: Igor Sapego

2020-01-28 Thread Igor Sapego
020 at 9:58 PM Ivan Pavlukhin > wrote: > > > Hello Ignite Community, > > > > The Project Management Committee (PMC) for Apache Ignite has invited > > Igor Sapego to join the PMC as a new member and we are pleased to > > announce that he has accepted. >

Re: .NET Near Cache - new flag in NearCacheConfiguration.java?

2020-02-18 Thread Igor Sapego
Do you suggest to introduce it in general configuration? Why not introduce it only on platform side? Is there any .NET-specific configuration? Best Regards, Igor On Tue, Feb 18, 2020 at 1:10 AM Pavel Tupitsyn wrote: > Igniters, > > I'm working on .NET Near Cache feature [1] > (storing deserial

Re: [MTCGA]: new failures in builds [4974714] needs to be handled

2020-02-18 Thread Igor Sapego
By the way, some ODBC tests became flaky about the same time as well. May it be there is one root cause somewhere in SQL? Best Regards, Igor On Mon, Feb 17, 2020 at 9:36 PM Pavel Tupitsyn wrote: > I did a quick look some time ago, no idea what is going on, honestly. Tests > became flaky for no

Re: Java thin client errors handling

2020-03-03 Thread Igor Sapego
Hi, I believe, we definitely should not ignore this. Alexey, you are the author of this code. What do you think? Best Regards, Igor On Thu, Feb 27, 2020 at 3:56 PM Aleksandr Shapkin wrote: > Hello! > > > > I just noticed that the Java thin client throws the following internal > exceptions: >

Re: Ignite 2.8 documentation

2020-03-03 Thread Igor Sapego
That's right, only C++ and .NET clients have partition awareness Best Regards, Igor On Tue, Mar 3, 2020 at 5:02 PM Artem Budnikov wrote: > Hi everyone, > > Looks like the following line from the Ignite 2.8 release notes is a bit > of an overstatement and should be removed: > > > Added support

Re: Node.JS, PHP, Python API references for Ignite 2.8 release

2020-03-05 Thread Igor Sapego
? > > [1] > https://github.com/apache/ignite/blob/master/modules/platforms/README.txt > > On Wed, 26 Feb 2020 at 21:42, Denis Magda wrote: > > > > Igor Sapego, Igniters, > > > > I've been working on Ignite website improvements that will be introduced >

Re: Thin client: compute support

2020-03-27 Thread Igor Sapego
> > > > - > > > Denis > > > > > > > > > On Wed, Mar 25, 2020 at 5:47 AM Alex Plehanov > > > > wrote: > > > > > > > Hello guys. > > > > > > > > I've implemented PoC and created IEP [1]

Re: Do we have any Python experts in the community?

2020-03-31 Thread Igor Sapego
I'm not very experienced in Python, but if no one else wants to take a look, I'll do it. Best Regards, Igor On Sat, Mar 28, 2020 at 2:04 AM Denis Magda wrote: > Igniters, > > Is any of you is skillful enough to contribute improvements to our Python > thin client? For instance, that's one of th

Re: [DISCUSSION] Major changes in Ignite in 2020

2020-04-10 Thread Igor Sapego
My plan includes: * Cluster API for C++ thick client * Moving Python, Node.js and PHP thin clients in separate repositories and separate release cycles. * Partition Awareness for Python and Node.js thin clients * SQL for C++ thin client. Steven, There are some third-party golang clients AFAIK, fo

Moving python, php and node.js in separate repos and release cycles

2020-04-21 Thread Igor Sapego
Guys, It was discussed on the dev list a few times that it would be a good idea to move Python, Node.js and PHP thin clients to separate repos and separate release cycles. In short there are several arguments for that: 1. There are no dependencies on the core functionality so there is simply no

Re: Moving python, php and node.js in separate repos and release cycles

2020-04-22 Thread Igor Sapego
sitive effect > on > > > community growth. Since newcomer may want to fix a bug and later use > > result > > > in his/her production environment. > > > > > > вт, 21 апр. 2020 г. в 13:27, Alexey Zinoviev : > > > > > >> Agree with thes

Re: IEP-44 Thin Client Discovery

2020-04-27 Thread Igor Sapego
Great, this feature is long awaited. 1. I believe so. Since I've proposed Partition Awareness feature I was thinking about a way for clients to discover cluster nodes. 2. In my opinion a simple boolean flag is enough for the beginning. In future maybe we can add a node filter. This can be useful

Re: IEP-44 Thin Client Discovery

2020-04-27 Thread Igor Sapego
t; client endpoints over 10.0.0.0/24 as well. > > > > We have to conform with IgniteConfiguration.LocalHost setting. > > If it is not set, or set to 0.0.0.0, we should gather IPs from all > > interfaces. > > But if it is set to something, we should gather only matching

Re: IEP-44 Thin Client Discovery

2020-04-28 Thread Igor Sapego
I guess it makes sense. If anyone needs more control over connection we would need to implement a new feature anyway (like node filter we discussed earlier) Best Regards, Igor On Tue, Apr 28, 2020 at 12:29 PM Pavel Tupitsyn wrote: > > enable the capability if the best effort affinity is on > I

Re: IEP-44 Thin Client Discovery

2020-05-07 Thread Igor Sapego
; > > > Ok, I've updated IEP and POC accordingly: > > > * Config flag removed > > > * IPs and host names retrieval simplified - use existing node > properties > > > and attributes instead of Compute > > > > > > On Tue, Apr 28, 2020 at 7:57 PM Ig

Re: [DISCUSSION] Ignite.C++ and CMake

2020-05-26 Thread Igor Sapego
Great! Let's start with creating a TC suite for it. The only concern I have is that it is one more build system to support. Should we get rid of autotools in 3.0? Best Regards, Igor On Tue, May 26, 2020 at 2:44 PM Alexey Kukushkin wrote: > +1. I recently completed a cross-IDE (MS Visual Stud

Re: [DISCUSSION] Ignite.C++ and CMake

2020-05-26 Thread Igor Sapego
e, so even this step (generation of project) is no > more necessary. > > > On 26.05.2020 16:02, Igor Sapego wrote: > > Great! > > > > Let's start with creating a TC suite for it. > > > > The only concern I have is that it is one more build system >

Re: IEP-50 Thin Client Continuous Queries

2020-07-10 Thread Igor Sapego
Pavel, What's about "stop" message? How can user unsubscribe from receiving notifications? Also, I believe I've seen discussions on removing initial query from continuous queries, as there are not any guarantees about getting consistent results with them. Should we avoid adding them in thin proto

Re: IEP-50 Thin Client Continuous Queries

2020-07-10 Thread Igor Sapego
m continuous queries > Interesting, I'm not aware of this. Can you please link those discussions? > > On Fri, Jul 10, 2020 at 2:04 PM Igor Sapego wrote: > > > Pavel, > > > > What's about "stop" message? How can user unsubscribe from receiv

Re: IEP-50 Thin Client Continuous Queries

2020-07-16 Thread Igor Sapego
icates issue with the initial query, but I > did > > > not > > > >> give it a second thought. > > > >> > > > >> Now I see that Vladimir was right - Initial query seems to be > > pointless, > > > >> since users can > > > >>

Node.js, Python, PHP thin clients place in release cycle

2020-08-07 Thread Igor Sapego
Guys, Currently, Node.js, Python and PHP thin clients are not included in Ignite release process, meaning we do not publish them on pip, npm, etc during release and do not publish API references for these clients. I propose to add steps to build and publish these client packages and API documenta

Re: Any reason to keep ClientConfiguration final?

2020-08-24 Thread Igor Sapego
t weird to me, can you please describe it > in more detail? > > Thanks, > Pavel > > On Mon, Aug 24, 2020 at 10:47 PM Denis Magda wrote: > >> @Pavel Tupitsyn , @Igor Sapego >> , >> >> Michael has been integrating Ignite with Micronaut and we hit some >>

Re: Ignite thin client in Rust

2020-08-24 Thread Igor Sapego
Hi, Val, I've been working on my implementation for some time, but didn't commit to it lately so it's pretty much abandoned. Maybe we should join our forces here :) Best Regards, Igor On Thu, Jul 2, 2020 at 1:00 AM Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Pavel, > > Yes, I

Re: Cpp thin client transactions support ready for review.

2020-08-24 Thread Igor Sapego
Great, I'll take a look. Best Regards, Igor On Mon, Aug 24, 2020 at 8:33 AM Zhenya Stanilovsky wrote: > > > Thanks Ivan Daschinsky for review, does anyone more who could check it ? > > thanks ! > >Igniters, seems i complete with transactions in thin cpp client > >implementation [1], part of ie

Re: [DISCUSSION] Consistency across java thin/thick APIs

2020-08-24 Thread Igor Sapego
Yes, it was an attempt to separate thick and thin clients as much as possible to move them in separate libs in future. Alex, what do you think? What is the right path here from the Java developer viewpoint? Best Regards, Igor On Mon, Aug 24, 2020 at 9:40 AM Alex Plehanov wrote: > Hi Val, > >

Re: IEP-51: Java Thin Client Async API

2020-08-24 Thread Igor Sapego
Alexey, what do you think? Which Future should be used here? Now, about the "not fully sync" interface - I believe this is acceptable as a first approach. Best Regards, Igor On Mon, Aug 24, 2020 at 12:37 PM Pavel Tupitsyn wrote: > I've changed the IEP and added a new future interface to the P

Re: Thin Client ping operation?

2020-09-14 Thread Igor Sapego
Николай, It looks a little bit hacky to me. I believe SQL drivers usually use that approach as a workaround because there is no other common way to do that. Sure we can recommend users to use cache.size() or anything other similar way to ensure the connection is alive, but it still looks like a w

Re: Thin Client ping operation?

2020-09-15 Thread Igor Sapego
gt; > > On the other hand, dedicated `ping` operation makes our API heavier > > > without adding new feature - > > > We can do the same with the other part of the API. > > > > > > Moreover, response to the ping doesn’t mean that SQL or cache query can > &g

Re: cpp thin client vector resize

2020-09-29 Thread Igor Sapego
Hi, Can you share your ignite::binary::BinaryType::Read method where reading of the std::vector is going on? Also, are your strings large too or only vectors? Best Regards, Igor On Mon, Sep 28, 2020 at 8:29 PM Brett Elliott wrote: > Hello, > > Tl;dr: I'm doing some profiling, and the cpp thi

Re: cpp thin client vector resize

2020-09-29 Thread Igor Sapego
, Igor On Tue, Sep 29, 2020 at 11:22 AM Igor Sapego wrote: > Hi, > > Can you share your ignite::binary::BinaryType::Read method where reading of > the std::vector is going on? > > Also, are your strings large too or only vectors? > > Best Regards, > Igor > > > O

Re: [EXTERNAL] Re: cpp thin client vector resize

2020-09-30 Thread Igor Sapego
m is my own fault. Thanks for > making me have a look at that again. > > Thanks, > Brett > > -Original Message- > From: Igor Sapego [mailto:isap...@apache.org] > Sent: Tuesday, September 29, 2020 2:22 AM > To: dev > Subject: [EXTERNAL] Re: cpp thin client vector

Re: [DISCUSS] Use GridNioServer in Java thin client

2020-11-09 Thread Igor Sapego
Sounds like a good idea to me. Best Regards, Igor On Mon, Nov 9, 2020 at 3:32 PM Alex Plehanov wrote: > +1 for using GridNioServer as java thin client communication layer. > > вс, 8 нояб. 2020 г. в 19:12, Pavel Tupitsyn : > > > Igniters, > > > > This is a continuation of "Use Netty for Java th

Re: IEP-54: Schema-first approach for 3.0

2020-11-24 Thread Igor Sapego
Pavel, I totally support that. Also, if we are aiming for stronger platform-independance, in our schemas we may want to support bit-notation (int32, uint64)? For example "long" can mean a different type on different platforms and it's easy to confuse them (happens often when using ODBC for example

Re: IEP-54: Schema-first approach for 3.0

2020-11-24 Thread Igor Sapego
gt; > > > > > > > > > On Tue, Nov 24, 2020 at 2:04 PM Pavel Tupitsyn > > wrote: > > > > > Agree, let's get rid of "long, short, byte" in the protocol definition. > > > > > > We can use Rust style, which is concise and

Re: [DISCUSS] Use GridNioServer in Java thin client

2020-12-04 Thread Igor Sapego
O > > > [2] https://github.com/apache/ignite/pull/8483 > > > > > > On Mon, Nov 9, 2020 at 4:07 PM Ivan Daschinsky > > > wrote: > > > > > >> I suppose that the best variant -- ability to switch to netty if this > > lib > > >> is in

  1   2   3   4   5   6   7   >