Re: JavaDoc for Event's subjectId methods

2019-09-30 Thread Denis Garus
Thank you, Maksim! I think that the description should look like: "Gets security subject ID initiated this task event if IgniteSecurity is enabled, otherwise returns null. This property is not available for GridEventType#EVT_TASK_SESSION_ATTR_SET task event." Please pay close attention that descr

Re: JavaDoc for Event's subjectId methods

2019-09-30 Thread Maksim Stepachev
Denis, I added it in my ticket and pull-request. Should I change the only first sentence or full comment? пн, 30 сент. 2019 г. в 11:27, Denis Garus : > Hello! > > I suggested to Maksim Stepachev include these changes in the scope of your > thicket [1] > and it looks like he agreed [2]. > > Maksi

Re: JavaDoc for Event's subjectId methods

2019-09-30 Thread Denis Garus
Hello! I suggested to Maksim Stepachev include these changes in the scope of your thicket [1] and it looks like he agreed [2]. Maksim Stepachev, could you please reflect JavaDoc and behavior changes of events in your ticket? 1. https://issues.apache.org/jira/browse/IGNITE-11992 2. http://apache-

Re: JavaDoc for Event's subjectId methods

2019-09-30 Thread Ivan Pavlukhin
Hi, Do we allow commits to master without a ticket? I can imagine only reverts as an exception. Otherwise a ticket is a primary process item. Work description, review, CI checks (we have a job checking javadocs). ср, 25 сент. 2019 г. в 01:15, Denis Magda : > > Denis, please feel free to go and e

Re: JavaDoc for Event's subjectId methods

2019-09-24 Thread Denis Magda
Denis, please feel free to go and edit the JavaDocs in place without a ticket. The changes suggested by you are reasonable. - Denis On Tue, Sep 24, 2019 at 3:55 AM Denis Garus wrote: > Hello, Igniters! > > Some events contain the subjectId method, for example, TaskEvent#subjectId. > The JavaDo

JavaDoc for Event's subjectId methods

2019-09-24 Thread Denis Garus
Hello, Igniters! Some events contain the subjectId method, for example, TaskEvent#subjectId. The JavaDoc for this method is: "Gets security subject ID initiated this task event, if available. This property is not available for GridEventType#EVT_TASK_SESSION_ATTR_SET task event. Subject ID will be