Re: Azure Cloud IP Finder

2021-04-08 Thread Atri Sharma
Hi Ilya, Please let me know if I can help with any further iterations on the PR. Regards, Atri On Wed, Apr 7, 2021 at 5:04 PM Atri Sharma wrote: > > Hi Ilya, > > Thanks for taking a look. I was able to resolve dependencies (Thanks, > Sam!) and have updated the PR. > > Copying the jars from

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

2021-04-08 Thread dpavlov . tasks
Hi Igniters, I've detected some new issue on TeamCity to be handled. You are more than welcomed to help. *Test with high flaky rate in master LongDestroyDurableBackgroundTaskTest.testLongIndexDeletionWithRebalance

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

2021-04-08 Thread dpavlov . tasks
Hi Igniters, I've detected some new issue on TeamCity to be handled. You are more than welcomed to help. *New Critical Failure in master Control Utility https://ci.ignite.apache.org/buildConfiguration/IgniteTests24Java8_ControlUtility?branch=%3Cdefault%3E No changes in the build

Re: Model of permissions for Ignite 3

2021-04-08 Thread Andrey Kuznetsov
Hi Denis! The idea and prototype look great. I'd like to highlight one arguable point. Default authorization implementation still assumes there are permissions provided in SecuritySubject. In turn, authentication is still responsible for filling these permissions. I suggest decoupling

[jira] [Created] (IGNITE-14508) Calcite integration: introduce SQL logical test runner

2021-04-08 Thread Taras Ledkov (Jira)
Taras Ledkov created IGNITE-14508: - Summary: Calcite integration: introduce SQL logical test runner Key: IGNITE-14508 URL: https://issues.apache.org/jira/browse/IGNITE-14508 Project: Ignite

Re: Model of permissions for Ignite 3

2021-04-08 Thread Denis Garus
Sorry, I forgot to point the link 1. https://github.com/apache/ignite/pull/8989 чт, 8 апр. 2021 г. в 17:50, Denis Garus : > Hello, Igniters! > > I want to propose to improve the way which we use > to present permissions in Ignite 3. > > The model of permission in Ignite has a set of drawbacks.

Model of permissions for Ignite 3

2021-04-08 Thread Denis Garus
Hello, Igniters! I want to propose to improve the way which we use to present permissions in Ignite 3. The model of permission in Ignite has a set of drawbacks. The main drawback, IMHO: if you need to add a new permission, you should change the core module by extended the 'SecurityPermission'

[jira] [Created] (IGNITE-14507) Un-deprecate IGNITE_BINARY_SORT_OBJECT_FIELDS

2021-04-08 Thread Stanislav Lukyanov (Jira)
Stanislav Lukyanov created IGNITE-14507: --- Summary: Un-deprecate IGNITE_BINARY_SORT_OBJECT_FIELDS Key: IGNITE-14507 URL: https://issues.apache.org/jira/browse/IGNITE-14507 Project: Ignite

welcome letter

2021-04-08 Thread Rodion Smolnikov
Hello Ignite Community! My name is Rodion. I want to contribute to Apache Ignite and want to start with this issue - IGNITE-14474, my JIRA username Smolnikov. Any help on this will be appreciated. Thanks! -- Спасибо, Родион Смольников Best regard, Rodion Smolnikov

Re: Terms clarification and modules splitting logic

2021-04-08 Thread Alexei Scherbakov
Alexey, Thanks for the detailed explanation. Ok, let's agree on having the internal package. I've created the ticket [1] to unify it's usage within the project. [1] https://issues.apache.org/jira/browse/IGNITE-14506 чт, 8 апр. 2021 г. в 15:34, Alexey Goncharuk : > Alexei, > > The main benefit

[jira] [Created] (IGNITE-14506) Use internal package for all Ignite's code.

2021-04-08 Thread Alexey Scherbakov (Jira)
Alexey Scherbakov created IGNITE-14506: -- Summary: Use internal package for all Ignite's code. Key: IGNITE-14506 URL: https://issues.apache.org/jira/browse/IGNITE-14506 Project: Ignite

Re: [DISCUSS] IEP-71 Public API for secondary index search

2021-04-08 Thread Maksim Timonin
Hi, Andrey! >> ScanQuery, TextQuery and partially SQL query share the same infrastructure I think I understand what you mean. I debug query processing and now agree that it's a nice idea to try to reuse the infrastructure of scan and text queries. Also as I can see there already Reducer

[jira] [Created] (IGNITE-14505) Print information about striped pool in metrics for local node

2021-04-08 Thread Denis Mekhanikov (Jira)
Denis Mekhanikov created IGNITE-14505: - Summary: Print information about striped pool in metrics for local node Key: IGNITE-14505 URL: https://issues.apache.org/jira/browse/IGNITE-14505 Project:

Re: Terms clarification and modules splitting logic

2021-04-08 Thread Alexey Goncharuk
Alexei, The main benefit from Jigsaw that I see for the project structure is controllable module interaction. Let's take our networking module as an example first. We may want to make sure that module implementation specifics do not leak to outside modules, so we define in the module definition

[DISCUSSION] Release 0.4.0 version of ignite python thin client (pyignite)

2021-04-08 Thread Ivan Daschinsky
Igniters! I suppose it is time to finally release new version of python thin client Recently, multiple bug fixes, features and other goodies have been implemented. For example: 1. Partition awareness. 2. Asyncio support. 3. Cluster API support. 4. Multiple performance improvements, especially for

Re: [DISSCUSSION] Common logger interface.

2021-04-08 Thread Alexei Scherbakov
Andrey, The PR looks good to me. Maybe we can wrap all internal threads into IgniteThread - I'm almost sure this will work in this way. Is it really need to use thread-locals for user threads? - probably not. I'm not sure if there is any problem at all. As soon as we want to have async API

[jira] [Created] (IGNITE-14504) ClusterGroupEmptyException: Cluster group is empty error after client reconnect

2021-04-08 Thread Andrey Aleksandrov (Jira)
Andrey Aleksandrov created IGNITE-14504: --- Summary: ClusterGroupEmptyException: Cluster group is empty error after client reconnect Key: IGNITE-14504 URL: https://issues.apache.org/jira/browse/IGNITE-14504

Re: [DISSCUSSION] Common logger interface.

2021-04-08 Thread Andrey Mashenkov
Also, with the suggested approach, we should avoid indirectly usage of ForkJoinPool internally or set our own pool instance explicitly when using reactive things. On Thu, Apr 8, 2021 at 1:33 PM Andrey Mashenkov wrote: > Alexey, > > I've made a PR for logger [1]. > Seems, we will need 2 logger

Re: [DISSCUSSION] Common logger interface.

2021-04-08 Thread Andrey Mashenkov
Alexey, I've made a PR for logger [1]. Seems, we will need 2 logger classes. 1. Node-aware logger adapter, that will add node prefix to messages and delegate calls to System.Logger or whatever. 2. Logger wrapper that will get logger from a thread-local. I don't like to use ThreadLocal directly

Re: [DISSCUSSION] Common logger interface.

2021-04-08 Thread Alexei Scherbakov
Andrey, *final* word in the example is missing, my bad. I like the static logger approach. Regarding your comments: * The static logger can easily be used by multiple nodes in a single JVM, it's a matter of implementation. It can be achieved by setting thread local logger context for the node.

[jira] [Created] (IGNITE-14503) Ability to specify project (fork) via the Version

2021-04-08 Thread Anton Vinogradov (Jira)
Anton Vinogradov created IGNITE-14503: - Summary: Ability to specify project (fork) via the Version Key: IGNITE-14503 URL: https://issues.apache.org/jira/browse/IGNITE-14503 Project: Ignite