Hello! Yes, I think this is a sensible approach.
Regards, -- Ilya Kasnacheev ср, 8 июл. 2020 г. в 14:46, Ivan Daschinsky <ivanda...@gmail.com>: > I think that this feature can be handled as compactFooter. For example, C++ > doesn't support compactFooter and it is not an issue. > Of course, this feature should be disabled by default, and should be > enabled explicitly in BinaryConfiguration. > Also, subsequent issues in jira about this feature support in platforms > should be created. > > ср, 8 июл. 2020 г. в 14:31, Ilya Kasnacheev <ilya.kasnach...@gmail.com>: > > > Hello! > > > > I think this is a blocker for this change. We already have binary format > > published: > > > > > https://apacheignite.readme.io/docs/binary-client-protocol-data-format#complex-object > > > > Arguably, we cannot change it in a minor version of Apache Ignite, so > this > > change has to target AI 3.0. > > > > Extending this binary format with e.g. new operations could probably be > OK. > > But we have clients released on a different schedule in their own repos > > (and there are some 3rd party clients too), we can't release a minor > > version which will change this format unilaterally without any change of > > operation (same data, same calls, different result after upgrade, broken > > clients). > > > > Regards, > > -- > > Ilya Kasnacheev > > > > > > ср, 8 июл. 2020 г. в 13:43, Ivan Daschinsky <ivanda...@gmail.com>: > > > > > Hi! > > > Ilya, unfortunatelly yes, subsequent changes should be made in C++, > .NET > > > and other platform code. > > > > > > ср, 8 июл. 2020 г. в 12:22, Ilya Kasnacheev <ilya.kasnach...@gmail.com > >: > > > > > > > Hello fellow devs, > > > > > > > > I just wanted to ask, how would this Binary Object format change > affect > > > > thin clients? C++/.Net nodes? Etc? > > > > > > > > Is it fully backwards compatible or not? > > > > > > > > I think that realistically, we can only add binary-incompatible > changes > > > to > > > > Binary Object format in 3.0. > > > > > > > > Regards, > > > > -- > > > > Ilya Kasnacheev > > > > > > > > > > > > ср, 8 июл. 2020 г. в 09:05, Ivan Pavlukhin <vololo...@gmail.com>: > > > > > > > > > A side note. Now we have a neat URL for TC bot > > > > > https://mtcga.ignite.apache.org/ (along with one in a gridgain > > > > > domain). > > > > > > > > > > 2020-07-07 18:43 GMT+03:00, Zhenya Stanilovsky > > > > <arzamas...@mail.ru.invalid > > > > > >: > > > > > > > > > > > > request it, check for example [1] > > > > > > > > > > > > also you need to run [2] tests. > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > > http://apache-ignite-developers.2346864.n4.nabble.com/Phani-Introduction-td47788.html > > > > > > [2] https://mtcga.gridgain.com > > > > > >>Hello, > > > > > >> > > > > > >>Look at the ticket and the only comment I can see is creating a > > > branch > > > > on > > > > > >>git in the main repo and not in my fork. I do not have the right > to > > > > > create > > > > > >> a > > > > > >>branch in the main repository. Am i missing something? > > > > > >> > > > > > >>Sorry I probably misread the document but I though that I should > > fork > > > > the > > > > > >>repo and then pull request as I do not have the rights to create > a > > > > > branch. > > > > > >> > > > > > >>Thanks for your help > > > > > >> > > > > > >> > > > > > >> > > > > > >>-- > > > > > >>Sent from: > http://apache-ignite-developers.2346864.n4.nabble.com/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > Best regards, > > > > > Ivan Pavlukhin > > > > > > > > > > > > > > > > > > -- > > > Sincerely yours, Ivan Daschinskiy > > > > > > > > -- > Sincerely yours, Ivan Daschinskiy >
