Re: IGNITE-13

2017-02-22 Thread Вадим Опольский
eString method. > 2. Run the benchmark with current implementation. > 3. Make the change described in the ticket. > 4. Run the benchmark with these changes. > 5. Compare results. > > Makes sense? Let me know if anything is unclear. > > -Val > > On Mon, Feb 20, 2017 at 8

Re: IGNITE-13

2017-02-20 Thread Вадим Опольский
;>> dsetrak...@apache.org> wrote: >>> >>>> On Thu, Feb 9, 2017 at 11:26 PM, Valentin Kulichenko < >>>> valentin.kuliche...@gmail.com> wrote: >>>> >>>> > Hi Vadim, >>>> > >>>> > I don't th

Re: assigning IGNITE-933

2017-02-24 Thread Вадим Опольский
nd propose > a fix. > > BTW, do you have any experience with Hibernate? If so, I would be amazing > if you pick up this ticket reassigning on yourself: > https://issues.apache.org/jira/browse/IGNITE-1794 < > https://issues.apache.org/jira/browse/IGNITE-1794> > > —

assigning IGNITE-933

2017-02-23 Thread Вадим Опольский
Dear sirs ! I want to resolve issue IGNITE-933 https://issues.apache.org/jira/browse/IGNITE-933 Is it actual ? In which class and method you want me to make changes ? Vadim Opolski

Re: IGNITE-13

2017-02-21 Thread Вадим Опольский
gt;>>> Vladimir, >>>>> >>>>> Can you please take a look and provide your thoughts? Can this be >>>>> applied to binary marshaller? From what I recall, it serializes string a >>>>> bit differently from optimized mar

Re: IGNITE-13

2017-02-15 Thread Вадим Опольский
ntin Kulichenko < >>>> valentin.kuliche...@gmail.com> wrote: >>>> >>>> > Hi Vadim, >>>> > >>>> > I don't think it makes much sense to invest into OptimizedMarshaller. >>>> > However, I would check if this optimizat

IGNITE-13

2017-02-09 Thread Вадим Опольский
Dear sirs! I want to resolve issue IGNITE-13 - https://issues.apache.org/jira/browse/IGNITE-13 Is it actual? Vadim Opolski

IGNITE-13

2017-02-09 Thread Вадим Опольский
Здравствуйте, коллеги! Это Опольский Вадим, я недавно в проекте и хочу решить тикет (IGNITE-13). Детализируйте пожалуйста, что конкретно нужно сделать? Где в проекте можно посмотреть примеры уже написанных more optimal marshaling ? Как сейчас маршализируются строки в проекте? Где описаны

Re: IGNITE-13

2017-03-01 Thread Вадим Опольский
rray. This should decrease memory consumption and > can also increase performance as we will avoid 'writeByteArray' step at > the end. > > Does it make sense to you? > > -Val > > On Mon, Feb 27, 2017 at 6:55 AM, Вадим Опольский <vaopols...@gmail.com> > wrote:

Fwd: Ignite ( IGNITE-3537 )

2016-12-16 Thread Вадим Опольский
транзакциями? Если я правильно понял, нужно написать юнит тесты. Вы не могли бы уточнить сколько их нужно написать? В какой папке их нужно писать? Вадим Опольский

Hello

2016-12-27 Thread Вадим Опольский
Dear sirs! I'm Vadim Opolskij, Jira user name is javaller. I want to contribute an ignite project, ticket number is IGNITE-3537. Vadim Opolskij

ready for review IGNITE-933

2017-03-24 Thread Вадим Опольский
ski 2017-03-21 11:30 GMT+03:00 Вадим Опольский <vaopols...@gmail.com>: > Hello everybody. > > The issue https://issues.apache.org/jira/browse/IGNITE-1794 fixed > originally by Mykola Pereyma. > I re-assigned issue https://issues.apache.org/jira/browse/IGNITE-1794 on > mys

Re: IGNITE - 4760 : working in hibernate module

2017-03-28 Thread Вадим Опольский
This is how you can easily prepare the dev env in IntellijIDEA: > https://cwiki.apache.org/confluence/display/IGNITE/Project+Setup < > https://cwiki.apache.org/confluence/display/IGNITE/Project+Setup> > > — > Denis > > > On Mar 27, 2017, at 7:14 AM, Вадим Ополь

Fwd: IGNITE - 4760

2017-03-24 Thread Вадим Опольский
Hello everybody, I want to resolve issue №4760 https://issues.apache.org/jira/browse/IGNITE-4760 To find solution I'm going to change method threadLocalForCache and to add Junit test. Why folder hibernate is not a module ? Can I added it ? Vadim Opolski

Re: IGNITE-4052 ready for review

2017-03-30 Thread Вадим Опольский
ith licence different from Apache Licence 2.0. Are you sure that for this test you need additional libs? I think this can be implemented without them.* *Also did you try to use new properties on real mesos cluster? Did work properly?* 2017-03-24 15:54 GMT+03:00 Вадим Опольский <vaopols...@gmail

IGNITE-4052 ready for review

2017-03-29 Thread Вадим Опольский
tribution. > > On Fri, Mar 24, 2017 at 3:54 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Nikolay, I will add properties for mesos role and unit test next week. >> >> -- Forwarded message -- >> From: Вадим Опольский <vaopols...@gmail.c

Re: IGNITE - 4760 : working in hibernate module

2017-03-27 Thread Вадим Опольский
gt; Vadim, > > ignite-hibernate module is a part of 'lgpl' profile. Apparently it's not > enabled in your environment. > > -Val > > On Fri, Mar 24, 2017 at 4:38 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > > > Hello everybody, > > > > I wa

IGNITE-4052 ready for review - what can I improve

2017-03-31 Thread Вадим Опольский
16:32 GMT+03:00 Вадим Опольский <vaopols...@gmail.com>: > Hello everyone! > > Nikolay, method Protos.FrameworkInfo.Builder#setRoleBytes and > Protos.FrameworkInfo.Builder#setUserBytes were added into > IgniteFrameworkInfoTest. > > Details please what do you want to

Fwd: IGNITE-4052 ready for review

2017-03-24 Thread Вадим Опольский
Nikolay, I will add properties for mesos role and unit test next week. -- Forwarded message -- From: Вадим Опольский <vaopols...@gmail.com> Date: 2017-03-22 15:53 GMT+03:00 Subject: Re: IGNITE-4052 ready for review To: dev@ignite.apache.org Nikolay, just changed status to

Re: IGNITE-4052 ready for review

2017-03-22 Thread Вадим Опольский
> > Thanks, > Nikolay > > On Wed, Mar 22, 2017 at 3:36 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > > > Hello everybody! > > > > Nikolay, > > review please https://github.com/apache/ignite/pull/1662 . > > > > Added ability

IGNITE-4052 ready for review

2017-03-22 Thread Вадим Опольский
Hello everybody! Nikolay, review please https://github.com/apache/ignite/pull/1662 . Added ability to configure current user parameters via system env properties - "MESOS_USER". Vadim Opolski -- Forwarded message ------ From: Вадим Опольский <vaopols...@gmail.com&g

Re: [jira] [Comment Edited] (IGNITE-4052) Add ability to set up users for MESOS

2017-04-03 Thread Вадим Опольский
Hello everybody! Nikolay, I didn't have experience with ignite-yarn integration and didn't have idea how fix it, but its interesting for me. Can you write name of another java application, in which I can see how they connect to mesos. Also, can you write how correctly to test working of

IGNITE - 4760 ready for review

2017-04-10 Thread Вадим Опольский
gt; ENTITY2_NAME regions. >> >> https://github.com/vadopolski/ignite/pull/1 >> >> Is it true? >> >> I have no idea how to change the method threadLocalForCache to support >> NONSTRICT_READ_WRITE strategy. I tried to change it in accordance with >> Cameroon Braid

Re: IGNITE-1794 is ready for review

2017-04-11 Thread Вадим Опольский
:00 Вадим Опольский <vaopols...@gmail.com>: > Dear sirs! > > Sorry for incorrect subject. > > https://github.com/apache/ignite/pull/1643 > > -- Forwarded message ------ > From: Вадим Опольский <vaopols...@gmail.com> > Date: 2017-03-24 15:48 GMT+0

Re: IGNITE-1794 is ready for review

2017-04-11 Thread Вадим Опольский
ignite-3477-master, it will be merged in master soon. > > Thanks > > On Tue, Apr 11, 2017 at 7:51 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > > > Hello guys! > > > > I added folder hibernate5 as module to project settings and discovered > some > > errors.

Re: IGNITE-1794 is ready for review

2017-04-11 Thread Вадим Опольский
> — > Denis > > > On Apr 11, 2017, at 9:51 AM, Вадим Опольский <vaopols...@gmail.com> > wrote: > > > > Hello guys! > > > > I added folder hibernate5 as module to project settings and discovered > some errors. Fixed errors in pull request - htt

Re: IGNITE - 4760 : working in hibernate module

2017-04-03 Thread Вадим Опольский
t; > This is how you can easily prepare the dev env in IntellijIDEA: > https://cwiki.apache.org/confluence/display/IGNITE/Project+Setup < > https://cwiki.apache.org/confluence/display/IGNITE/Project+Setup> > > — > Denis > > > On Mar 27, 2017, at 7:14 AM, Вадим Ополь

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

2017-04-12 Thread Вадим Опольский
Hello guys! Nikolay, I looked at cassandra mesos integration, but cant find how to test ability to configure mesos and user via system env properties. I will continue to search in another projects which use Mesos. Review please pull request - https://github.com/vadopolski/ignite/pull/2. It

Re: IGNITE - 4760 : working in hibernate module

2017-04-06 Thread Вадим Опольский
ust need to define multiple caches that will be mapped to > Hibernate regions and initiate a transaction that updates several regions > (caches) at once? > > — > Denis > > > On Apr 3, 2017, at 10:39 AM, Вадим Опольский <vaopols...@gmail.com> > wrote: &

Re: Apache Ignite 2.0 Release

2017-04-06 Thread Вадим Опольский
Hello community! Denis, issue IGNITE-1794 - Hibernate 5 Support has been ready for review from 24 March. The tests in Team City executed successfully. Vadim Opolski 2017-04-06 5:04 GMT+03:00 Denis Magda : > *Denis Magda:*

IGNITE-1794 is ready for review

2017-04-06 Thread Вадим Опольский
Dear sirs! Sorry for incorrect subject. https://github.com/apache/ignite/pull/1643 -- Forwarded message -- From: Вадим Опольский <vaopols...@gmail.com> Date: 2017-03-24 15:48 GMT+03:00 Subject: ready for review IGNITE-933 To: dev@ignite.apache.org, Denis Magda <dma...@a

Re: IGNITE - 4760 ready for review

2017-04-15 Thread Вадим Опольский
://github.com/apache/ignite/pull/1768 Vadim 2017-04-13 15:05 GMT+03:00 Semyon Boikov <sboi...@gridgain.com>: > Hi Vadim, > > I added comments in JIRA. I'll apply the same fix for hibernate5 when fix > is finalized. > > Thanks! > > On Thu, Apr 13, 2017 at 12:10 PM, Вадим О

IGNITE-4052 ready for review

2017-04-17 Thread Вадим Опольский
Hi, Nikolay! I've made the following improvements ( https://github.com/apache/ignite/pull/1783 ): 1) Moved framework builder related code to separate method, so it make code cleaner and framework testable. 2) Call framework builder method from test to test role and user. 3) Added validating

Re: IGNITE - 4760 ready for review

2017-04-13 Thread Вадим Опольский
myon > > On Mon, Apr 10, 2017 at 8:15 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Hello guys! >> >> Semyon, review please again. Test check corresponding IgniteCaches >> contain expected number of entries. Test fails for >> HibernateN

Re: [jira] [Updated] (IGNITE-4052) Add ability to set up users for MESOS

2017-04-13 Thread Вадим Опольский
khonov updated IGNITE-4052: > - > > Vadim, > > I've left comment in the ticket. > > On Wed, Apr 12, 2017 at 3:00 PM, Вадим Опольский <vaopols...@gmail.com> > > > > > Add ability to set up users for MESOS > > -

Re: IGNITE - 4760 ready for review

2017-04-13 Thread Вадим Опольский
will check code style rules. Vadim 2017-04-13 15:05 GMT+03:00 Semyon Boikov <sboi...@gridgain.com>: > Hi Vadim, > > I added comments in JIRA. I'll apply the same fix for hibernate5 when fix > is finalized. > > Thanks! > > On Thu, Apr 13, 2017 at 12:10 PM, Вадим Опольский

IGNITE-1084 and IGNITE-4760 are ready for review

2017-04-20 Thread Вадим Опольский
t > fix in pull/1828 is not correct. I added new comments in IGNITE-4760, > please take a look. > > Thanks, > Semyon > > On Wed, Apr 19, 2017 at 12:17 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Hello guys! >> >> The issue I

Fwd: IGNITE - 4760 ready for review

2017-04-19 Thread Вадим Опольский
Hi guys! Semyon, have you viewed https://github.com/apache/ignite/pull/1768 Is it true for issue https://issues.apache.org/jira/browse/IGNITE-4760 ? What can I improve ? Vadim Opolski -- Forwarded message -- From: Вадим Опольский <vaopols...@gmail.com> Date: 2017-04-15

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

2017-04-23 Thread Вадим Опольский
Hi guys! Nikolay, the next improvements are maden ( https://github.com/apache/ignite/pull/1783): - changed code according with code style (extra space, if else blocks, javadoc) - written two tests one which will testing new parametes and other which will testing role validation -

Re: IGNITE-13 (ready for review)

2017-03-09 Thread Вадим Опольский
LatchBenchmark.binaryHeapOutputStreamInDirect Test avgt 5048,506 ± 1,116 ns/op Vadim 2017-03-06 19:42 GMT+03:00 Вадим Опольский <vaopols...@gmail.com>: > Hello, everybody! > > Valentin, I've corrected benchmark and received the results: > >

IGNITE-933 ready for review

2017-03-14 Thread Вадим Опольский
Please prepare a pull-request and make sure that the test passes on > TeamCity. > > — > Denis > > > On Mar 10, 2017, at 9:25 AM, Вадим Опольский <vaopols...@gmail.com> > wrote: > > > > Hello everyone! > > > > Denis, I've successfully executed

Re: IGNITE-13 (ready for review)

2017-03-13 Thread Вадим Опольский
tin.kuliche...@gmail.com>: >> >>> Hi Vadim, >>> >>> Results are a bit confusing. Any idea why it's better on long string, >>> but worse on a short string? If that's actually the case, there is no any >>> reason to make the change and I would ju

Re: assigning IGNITE-933

2017-03-10 Thread Вадим Опольский
dma...@apache.org>: > Vadim, > > The timeout should be such that the test completes successfully on single > core machine as well as on multi-core servers. Setting timeout to 3 - 5 > seconds should work. > > — > Denis > > On Mar 7, 2017, at 1:55 AM, Вадим Опольский <

Re: IGNITE-13 (ready for review)

2017-03-10 Thread Вадим Опольский
on to make the change and I would just close the ticket. > > -Val > > On Thu, Mar 9, 2017 at 9:20 AM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Hello everyone! >> >> Colleagues, take a look please at the results of measuring. >> >&g

Re: IGNITE-13

2017-03-06 Thread Вадим Опольский
ength > while encoding and writing bytes, and then write length. > > -Val > > On Fri, Mar 3, 2017 at 12:45 AM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Valentin, >> >> What do you think about duplicated cycle in strToBinaryOutputStream ? &g

Re: assigning IGNITE-933

2017-03-07 Thread Вадим Опольский
ware. > > — > Denis > > On Mar 6, 2017, at 2:31 AM, Вадим Опольский <vaopols...@gmail.com> wrote: > > Hi everyone! > > I executed test GridFailFastNodeFailureDetectionSelfTest.testFailFast > fails periodically with different values of node quantity and awaiting &g

Re: assigning IGNITE-933

2017-03-06 Thread Вадим Опольский
s://issues.apache.org/jira/browse/IGNITE-1794 < > https://issues.apache.org/jira/browse/IGNITE-1794> > > — > Denis > > > On Feb 23, 2017, at 2:40 AM, Вадим Опольский <vaopols...@gmail.com> > wrote: > > > > Dear sirs ! > > > >

Re: IGNITE-13

2017-03-02 Thread Вадим Опольский
Your > current results are actually promising, but I would like to confirm them. > > -Val > > On Wed, Mar 1, 2017 at 6:17 AM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Hi Valentin! >> >> Thank you for comments. >> >> There is a new

Re: IGNITE-13

2017-03-03 Thread Вадим Опольский
actually make any > performance difference? If so, I think we can close the ticket. > > Vova, can you also take a look and provide your thoughts? > > -Val > > On Thu, Mar 2, 2017 at 1:27 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Hi Valentin! >&

Re: IGNITE-13

2017-03-03 Thread Вадим Опольский
that it didn't actually make any > performance difference? If so, I think we can close the ticket. > > Vova, can you also take a look and provide your thoughts? > > -Val > > On Thu, Mar 2, 2017 at 1:27 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> H

Re: IGNITE-4052 ready for review

2017-04-18 Thread Вадим Опольский
Hi Nikolay! The comments was added. Vadim 2017-04-17 15:32 GMT+03:00 Nikolai Tikhonov <ntikho...@apache.org>: > Vadim, > > Thank you for your contribution. I'll look at changes. Can you please post > the list improvements to jira ticket? > > On Mon, Apr 17, 2017 at

IGNITE-1084 is ready for review

2017-04-19 Thread Вадим Опольский
Hello guys! The issue IGNITE-1084 https://issues.apache.org/jira/browse/IGNITE-1794 fixed originally by Milap Wadhwa. I reassigned it on myself because she didn't make improvements from Anton Vinogradov's comments. Also I just finished another issue with hibernate's test. Review it please -

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

2017-05-15 Thread Вадим Опольский
Hello Prachi! I reassigned issue https://issues.apache.org/jira/browse/IGNITE-4052 to you to approve changes of page - https://apacheignite.readme.io/docs/mesos-deployment#section-ignite-mesos-framework Vadim 2017-05-11 11:54 GMT+03:00 Вадим Опольский <vaopols...@gmail.com>: > H

IGNITE-4447

2017-05-15 Thread Вадим Опольский
Hi guys! Vladimir, is issue https://issues.apache.org/jira/browse/IGNITE-4447 actual now? Which tests do u mean? Detail please. Vadim

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

2017-05-11 Thread Вадим Опольский
Hello guys! I want to update https://apacheignite.readme.io/docs/mesos-deployment with new text according with https://cwiki.apache.org/confluence/display/IGNITE/Documentation But have a problem with authentication, registered login vaopols...@gmail.com. Help me, please. New text: 3.Copy the

Re: IGNITE-1084 is ready for review

2017-05-10 Thread Вадим Опольский
828 is not correct. I added new comments in IGNITE-4760, > please take a look. > > Thanks, > Semyon > > On Wed, Apr 19, 2017 at 12:17 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > >> Hello guys! >> >> The issue IGNITE-1084 https://issues.apac

Scala issue

2017-05-10 Thread Вадим Опольский
Hi guys! I want to resolve ticket with Scala. Recommend me please such a issue. Maybe with Spark. Vadim Opolski

IGNITE - 3469

2017-05-18 Thread Вадим Опольский
Hi guys! Is issue https://issues.apache.org/jira/browse/IGNITE-3469 actual now ? What else needs to be done ? Vadim

Re: IGNITE-4447 ready for review

2017-05-17 Thread Вадим Опольский
check test pass on TeamCity in this suite, could you please provide > link for related TeamCity run? > > Thanks > > On Tue, May 16, 2017 at 1:35 PM, Вадим Опольский <vaopols...@gmail.com> > wrote: > > > Hi guys! > > > > IgniteCache150Clie

Fwd: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

2017-05-19 Thread Вадим Опольский
Hello Prachi! Have you seen the edits in readme.io for Mesos Deployment page: https://dash.readme.io/project/apacheignite/v2.0/suggested Vadim Opolski -- Forwarded message -- From: Denis Magda (JIRA) Date: 2017-05-15 23:02 GMT+03:00 Subject: [jira]

Re: [jira] [Commented] (IGNITE-4052) Add ability to set up users for MESOS

2017-05-23 Thread Вадим Опольский
Hello , Prachi! I reviewed reference. Removing occurrence it was the right action, Thank you! Vadim Opolski 2017-05-23 2:19 GMT+03:00 Prachi Garg (JIRA) : > > [ https://issues.apache.org/jira/browse/IGNITE-4052?page=com. >

Re: IGNITE-4447 ready for review

2017-05-18 Thread Вадим Опольский
myon Boikov <sboi...@gridgain.com>: > >> Hi Vadim, >> >> Did you check test pass on TeamCity in this suite, could you please >> provide >> link for related TeamCity run? >> >> Thanks >> >> On Tue, May 16, 2017 at 1:35 PM, Вадим Опольский <va

IGNITE-4756

2017-05-30 Thread Вадим Опольский
Hello guys! I want to resolve isuue - https://issues.apache.org/jira/browse/IGNITE-4756 But cant find information about where to create log message in correct format. Help please. Vadim Opolski

place for printing info about partition distribution

2017-05-31 Thread Вадим Опольский
Hello guys! I want to discuss and suggest some format and place for pinting info about partition distribution to log from issue https://issues.apache.org/jira/browse/IGNITE-4756 What do you think about it ? Issue is about RendezvousAffinityFunction and FairAffinityFunction classes. Vadim

Re: is IGNITE-3142 actual now

2017-06-07 Thread Вадим Опольский
OK, thank you, Denis! 2017-06-06 21:04 GMT+03:00 Denis Magda <dma...@apache.org>: > Vadim, > > Please try to run the reproducer attached. If the error pops up then the > bug needs to be fixed. > > — > Denis > > > On Jun 6, 2017, at 4:36 AM, Вадим Опол

Fwd: place for printing info about partition distribution (IGNITE-4756)

2017-06-04 Thread Вадим Опольский
Do you agree ? -- Forwarded message -- From: Вадим Опольский <vaopols...@gmail.com> Date: 2017-06-04 13:42 GMT+03:00 Subject: Fwd: place for printing info about partition distribution (IGNITE-4756) To: Denis Magda <dma...@apache.org>, tled...@gridgain.com, Yakov Zhdanov < yzhda...@gridgai

is IGNITE-3142 actual now

2017-06-06 Thread Вадим Опольский
Hello guys! I want to resolve issue https://issues.apache.org/jira/browse/IGNITE-3142 Is it actual now? Vadim Opolski