Re: IGNITE-14812: Statistics

2021-11-02 Thread Nikita Safonov
Hello Igniters, The PR with latest documentation on SQL statistics is here: https://github.com/apache/ignite/pull/9550 Regards, Nikita пн, 5 июл. 2021 г. в 12:53, Nikita Safonov : > Hello Igniters, > > I'd be glad to help with documenting this functionality. > I do believe that it won't take

Re: IGNITE-14812: Statistics

2021-07-05 Thread Nikita Safonov
Hello Igniters, I'd be glad to help with documenting this functionality. I do believe that it won't take long. Hope to finish it before the end of next week to have enough time for docs discussion. Regards, Nikita ср, 23 июн. 2021 г. в 18:11, Ivan Daschinsky : > AFAIK, it is enough to include

Re: IGNITE-14812: Statistics

2021-06-23 Thread Ivan Daschinsky
AFAIK, it is enough to include mention of this library in NOTICE, please see here [1][2][3] [1] -- https://www.apache.org/legal/src-headers.html#3party [2] -- https://www.apache.org/legal/src-headers.html#notice [3] -- https://www.apache.org/legal/resolved.html ср, 23 июн. 2021 г. в 17:36,

Re: IGNITE-14812: Statistics

2021-06-23 Thread Valentin Kulichenko
Taras, It is true that we try to minimize dependencies, but there are some anyway. I think it's perfectly fine to add this library as a dependency. Will you be able to do this asap? It is surely better than reverting the commit :) -Val On Wed, Jun 23, 2021 at 12:32 AM Taras Ledkov wrote: >

Re: IGNITE-14812: Statistics

2021-06-23 Thread Taras Ledkov
Hi, > Taras, can you, please, describe the features that was implemented in this merge? > How users supposed to use them? > Do we have plans to document? Sure. Alexander Belyak will describe and file ticket to documentation. On 23.06.2021 9:27, Nikolay Izhikov wrote: Hello, Taras. Thanks

Re: IGNITE-14812: Statistics

2021-06-23 Thread Nikolay Izhikov
Hello, Taras. Thanks for feedback. > AFAIK and as long as I can remember Ignite project try to minimize external > dependencies usage and adds new external dependency only when there is no > other way out. Does it mean we have to incapsulate every external library we want to use? Taras, can

Re: IGNITE-14812: Statistics

2021-06-23 Thread Taras Ledkov
Hi, We have discussed BCrypt include/add dependency here [1]. AFAIK and as long as I can remember Ignite project try to minimize external dependencies usage and adds new external dependency only when there is no other way out. [1].

Re: IGNITE-14812: Statistics

2021-06-22 Thread Valentin Kulichenko
Dmitry, As the PMC chair, would you mind contacting legal regarding the matter? This is not the only example of such code (e.g. [1]), so we should look into this asap. [1] https://github.com/apache/ignite/blob/master/modules/core/src/main/java/org/mindrot/BCrypt.java As for this particular

Re: IGNITE-14812: Statistics

2021-06-22 Thread Dmitry Pavlov
Hi Nikolai, thank you for noticing. I guess it's not about license, but about Intellectual Property (IP) ownership. AFAIK, Apache License 2.0 is here and AL 2.0 is definetely allowed to be used in the codebase for an Apache project (https://www.apache.org/legal/resolved.html) But licenese

IGNITE-14812: Statistics

2021-06-22 Thread Nikolay Izhikov
Hello, Igniters. Recently huge commit was merged [1]. Taras, Alexander, can you, please, explain what is purpose of the commit? What feature it implemented? Looked inside the ticket and found no explanation. Description is "Add statistics collection and usage.» Do we have plans to document