Hi, Pavel. > Which tasks are required for control.sh?
All management tasks inherit from the `VisorMultiNodeTask` class. > Should we add a special case for them instead of enabling all tasks by > default? We can do it this way, but then it is possible to execute any task in the Ignite classpath that inherits from the management task class. This is necessary for pluggable commands - in Ignite extensions and plugins. In any case, to avoid security issues it seems the user should configure an explicit security plugin - which has the functionality needed for this. On Fri, Nov 8, 2024 at 4:15 PM Pavel Tupitsyn <ptupit...@apache.org> wrote: > > I agree with the proposal in general, but this part is concerning: > > > Change the default parameter - > ThinClientConfiguration#setMaxActiveComputeTasksPerConnection (enable tasks > execution by default) > > This might become a security issue. > As a user, I might want to disable thin client compute tasks but still use > the control utility. > > Which tasks are required for control.sh? Should we add a special case for > them instead of enabling all tasks by default? > > On Fri, Nov 8, 2024 at 3:00 PM Nikolay Izhikov <nizhi...@apache.org> wrote: > > > +1 > > > > > On 8 Nov 2024, at 08:26, Nikita Amelchev <namelc...@apache.org> wrote: > > > > > > Hello, Igniters. > > > > > > I propose to migrate the control.sh utility to a thin client. This > > > will allow us to: > > > > > > - Remove the legacy GridClient, which is used only for control.sh; > > > - Developers will support one thin client, instead of two; > > > - Simplify cluster configuration. > > > > > > Implementation details and migration notes are described in the IEP > > > [1]. PRs are ready to review [2,3,4]. Please, take a look. > > > > > > [1] > > https://cwiki.apache.org/confluence/display/IGNITE/%5BPhase+3%5D+Use+IgniteClient+instead+of+GridClient+in+the+control.sh+utility > > > [2] https://github.com/apache/ignite/pull/11647 > > > [3] https://github.com/apache/ignite/pull/11646 > > > [4] https://github.com/apache/ignite/pull/11648 > > > > > > > > > -- > > > Best wishes, > > > Amelchev Nikita > > > > -- Best wishes, Amelchev Nikita
