Re: Internal classes are exposed in public API

2020-01-30 Thread Anton Vinogradov
>> . Remove @Deprecated from old API (because it strange to have one >> deprecated API and second experimental API) >> 2. Add @IgniteExperimetnal to new API (because... see item. 1) -1 We should have all the options. Deprecated -> will be removed at major release (because this way is slow or

Re: Excessive backups performance suggestion.

2020-02-06 Thread Anton Vinogradov
The statement seems to be correct for cases when we'd like to perform fast loading and computation and not afraid of data loss. Performance boost always means we lose some guarantee. Let's just update the proposal with the explicit warning that reducing backups to zero may lead to data loss on any

Re: [REVIEW REQUEST] IGNITE-12630 Remove developers sections from parent pom.xml

2020-02-06 Thread Anton Vinogradov
Looks good to me. On Thu, Feb 6, 2020 at 11:45 AM Ivan Pavlukhin wrote: > Igniters, > > I raised a PR for a ticket [1] removing section from > parent pom.xml. I described the motivation in the ticket. Shortly, > this section has a little meaning today and even worse is misleading. > > Please

Re: Excessive backups performance suggestion.

2020-02-06 Thread Anton Vinogradov
> I'm quite sure most users do not want to lose data by default. > > It's not wise to recommend such thing even with explicit warning. > > чт, 6 февр. 2020 г. в 13:30, Anton Vinogradov : > > > The statement seems to be correct for cases when we'd like to perform > fast > &

Re: Forbid mixed cache groups with both atomic and transactional caches

2020-02-04 Thread Anton Vinogradov
Seems, we already started the separation by atomic operations restriction inside the transactions [1]. See no reason to allow mixes in this case. [1] https://issues.apache.org/jira/browse/IGNITE-2313 On Tue, Feb 4, 2020 at 2:28 PM Ivan Rakov wrote: > Igniters, > > Apparently it's possible in

Re: [VOTE] Allow or prohibit a joint use of @deprecated and @IgniteExperimental

2020-02-10 Thread Anton Vinogradov
-1 Prohibit On Mon, Feb 10, 2020 at 5:30 PM Alexey Kuznetsov wrote: > -1 Prohibit > > From my point of view, we should not deprecate the old API if the new API > is marked as experemental. > > > On Mon, Feb 10, 2020 at 4:47 PM Konstantin Orlov > wrote: > > > -1 Prohibit > > > > We should not

Partition reserve/release asymmetry

2020-01-08 Thread Anton Vinogradov
at release attempt? 2) Any objection to merging assertion instead of weird return to the master branch? 3) Any Idea why the exception happens? [1] https://gist.githubusercontent.com/anton-vinogradov/834fc63114a3e8d46b89ea4ccec8148b/raw/6438930c7fef119d0ad60df76d821fe7bd100c5e/gistfile1.txt [2] https

Re: Master compilation error

2020-01-20 Thread Anton Vinogradov
Also, is it possible to perform the same check on the merge attempt? Each PR can be merged from GitHub page, can we append check to the "megre button" flow? Can we restrict merges bypassing this button? On Mon, Jan 20, 2020 at 2:17 PM Anton Vinogradov wrote: > Should we perform

Re: Master compilation error

2020-01-20 Thread Anton Vinogradov
gt; "Visa" truly works only when master is in the same state during the merge > as it was during TC run. > > On Mon, Jan 20, 2020 at 1:50 PM Anton Vinogradov wrote: > > > It seems, this because of IGNITE-12227 fix [1]. > > Main question is "how this may happen

Re: Internal classes are exposed in public API

2020-01-20 Thread Anton Vinogradov
+1 to @IgniteExperimental On Mon, Jan 20, 2020 at 12:09 PM Alexey Goncharuk < alexey.goncha...@gmail.com> wrote: > After giving it some thought, I agree with Denis - there is nothing wrong > with exposing the new APIs, we just need to make it clear that we are still > going to change it. > >

Re: Master compilation error

2020-01-20 Thread Anton Vinogradov
It seems, this because of IGNITE-12227 fix [1]. Main question is "how this may happen in case fix got the Visa [2] ?". [1] https://ci.ignite.apache.org/viewModification.html?modId=895719=false=IgniteTests24Java8_BuildApacheIgnite=vcsModificationFiles [2]

Re: Apache Ignite 2.8 RELEASE [Time, Scope, Manager]

2020-01-22 Thread Anton Vinogradov
Good Idea, this will also check that the release process is alive. On Wed, Jan 22, 2020 at 12:04 PM Alexey Goncharuk < alexey.goncha...@gmail.com> wrote: > Folks, Maxim, > > Do you mind if I build the current state of ignite-2.8 branch and upload a > maven staging as rc0 (step 4.3.2 of the

Re: Partition reserve/release asymmetry

2020-01-10 Thread Anton Vinogradov
Everything is fine. Merged to master branch. On Fri, Jan 10, 2020 at 9:48 AM Anton Vinogradov wrote: > >> Does the issue reproduce in > >> subsequent runs? > Unfortunately no. > We performed 30+ runs without "success". > > >> I think we can add an a

Re: Partition reserve/release asymmetry

2020-01-09 Thread Anton Vinogradov
t; eviction/renting code looks overly complicated, perhaps, there is a bug > somewhere there? I think we can add an assertion to > GridDhtLocalPartition#destroy() method to check that reservations is 0 when > this method is called (there is a check for EVICTED state already there) > > --

Re: PME speedup #2, TX recovery delay elimination.

2019-12-26 Thread Anton Vinogradov
des leaves almost at the same > time (perhaps due to some network connectivity issues). With a delay > recovering multiple failed nodes will be grouped into one recovery > round (+PME). Correct me if my understanding is wrong. Was there any > performance measurements for such multiple left nod

Re: PME speedup #2, TX recovery delay elimination.

2019-12-24 Thread Anton Vinogradov
Rechecked TC two more times. Going to merge to master in case no objections here. On Mon, Dec 23, 2019 at 1:44 PM Anton Vinogradov wrote: > Igniters, > > One more PME optimization ready to be reviewed. > I found a strange tx recovery delay caused by IGNITE_TX_SALVAGE_TIMEOUT. &g

Re: Ignite 2.8 announcement plan

2020-03-11 Thread Anton Vinogradov
ailed node was in the baseline topology? How > about pure in-memory clusters and clusters with CacheStore? > > - > Denis > > > On Tue, Mar 10, 2020 at 12:40 AM Anton Vinogradov wrote: > > > Denis, > > > > >> the blocking PME no longer happens if a

Re: Ignite 2.8 documentation

2020-03-11 Thread Anton Vinogradov
Artem, I've updated the Read Repair page On Thu, Mar 5, 2020 at 3:51 PM Artem Budnikov wrote: > Anton, > > Could you please review the page about Read Rapair? > > https://apacheignite.readme.io/docs/read-repair > > -Artem > > On 05.03.2020 12:20, Artem Budnikov wrote: > > OK, I'll recreate it.

Re: Ignite 2.8 announcement plan

2020-03-10 Thread Anton Vinogradov
Please feel free to share feedback before > it's published. You can use the comment feature of Google Docs: > > https://docs.google.com/document/d/1ssTC1Jf_reqZFWgl4ayhaohiAJCpzdL4tPrHTxvfvAM/edit?usp=sharing > > > @Alexey Zinoviev , @Andrey Gura > , @Nikolay Izhikov , @Anto

Active nodes aliveness WatchDog

2020-04-08 Thread Anton Vinogradov
Igniters, Do we have some feature allows to check nodes aliveness on a regular basis? Scenario: Precondition The cluster has no load but some node's JVM crashed. Expected actual The user performs an operation (eg. cache put) related to this node (via another node) and waits for some timeout

Re: Active nodes aliveness WatchDog

2020-04-08 Thread Anton Vinogradov
more complex… like there’s the partition loss policy and > rebalancing doesn’t always happen (configurable, persistence, etc)… but > broadly it does as you expect. > > Regards, > Stephen > > > On 8 Apr 2020, at 08:40, Anton Vinogradov wrote: > > > > Ignite

Re: Active nodes aliveness WatchDog

2020-04-08 Thread Anton Vinogradov
/ignite.apache.org/releases/latest/javadoc/org/apache/ignite/spi/discovery/tcp/TcpDiscoverySpi.html > > Other people would be better placed to discuss the internals. > > Regards. > Stephen > > > On 8 Apr 2020, at 09:32, Anton Vinogradov wrote: > > > > Stephe

Re: Active nodes aliveness WatchDog

2020-04-08 Thread Anton Vinogradov
gnite/blob/e9b3c4cebaecbeec9fa51bd6ec32a879fb89948a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/messages/TcpDiscoveryStatusCheckMessage.java > > > > Regards, > > Stephen > > > >> On 8 Apr 2020, at 10:04, Anton Vinogradov wrote: > >> >

Re: Inconsistent API IgniteClient and REST

2020-04-01 Thread Anton Vinogradov
Folks, The question is not about "+1" or "-1". The question is "why?". This looks like some special feature to solve some special security issue but may be just a bad/broken/obsolete/unrefactored code. Changing this semantic we'll fix bad code or break some contract. 50% to 50%. Let's keep REST

Re: Inconsistent API IgniteClient and REST

2020-04-01 Thread Anton Vinogradov
llo, Anton. > > What is «contract» for you? > Do we have this contract written somewhere? > > > > 1 апр. 2020 г., в 11:35, Anton Vinogradov написал(а): > > > > Folks, > > > > The question is not about "+1" or "-1". > > The ques

Get rid of @Nullable and @NotNull

2020-03-27 Thread Anton Vinogradov
Folks, Found we still use @Nullable annotation. What's the reason for using it? Everything is Object and Nullable :) How about get rid of @Nullable usages and restrict its usage by checkstyle plugin? BTW, We already "do not use @NotNull annotation" (с) Coding Guidelines [1] which may have some

Re: Get rid of @Nullable and @NotNull

2020-03-27 Thread Anton Vinogradov
gt; > > > > > Intellij idea IDE has a static code analysis, which uses that > > > > annotation too. IDE highlights possible problems. It helps to make > > > > our code more stable and bugless. > > > > > > > > пт, 27 мар. 2020 г. в 12:06, Pa

Re: [DISCUSSION] `static final` code-style constant naming rules

2020-04-27 Thread Anton Vinogradov
>> +1 to force upper case for `static final` variables. +1 too On Mon, Apr 27, 2020 at 12:08 PM Nikolay Izhikov wrote: > +1 to force upper case for `static final` variables. > > > 25 апр. 2020 г., в 07:39, Ivan Pavlukhin > написал(а): > > > > Maxim, > > > > Thank you for efforts in a code

Re: [DISCUSSION] Major changes in Ignite in 2020

2020-04-28 Thread Anton Vinogradov
Folks, I keep working on crash recovery speed-up. The main goal is to have put/get operations latency less than 500 ms on node fail/left. Currently, latency can be increased to seconds or even dozens of seconds. The task is split to 2 threads - Switch and tx recovery speed-up. Speed-up can be

Re: [DISCUSSION] Major changes in Ignite in 2020

2020-04-30 Thread Anton Vinogradov
ton, > > Thanks for sharing the plans with details. Could you please do us a favor > and add this item to the roadmap page? I will also help if you link the > item to a JIRA ticket or IEP that includes the details you shared here. > > - > Denis > > > On Tue, Apr 28,

Re: [DISCUSS] Data loss handling improvements

2020-05-07 Thread Anton Vinogradov
at 8:02 PM Alexei Scherbakov < alexey.scherbak...@gmail.com> wrote: > ср, 6 мая 2020 г. в 12:54, Anton Vinogradov : > > > Alexei, > > > > 1,2,4,5 - looks good to me, no objections here. > > > > >> 3. Lost state is impossible to reset if a topology doesn'

Re: Crash recovery speed-up #3, Cellular Switch

2020-05-14 Thread Anton Vinogradov
Folks, It seems, we have tacit agreement here. Going to merge fix May 15. On Tue, May 12, 2020 at 10:08 AM Anton Vinogradov wrote: > Denis, > > Rebalance is not expected here since this optimization works only on a > fully rebalanced cluster with baseline. > > On Sat, May 9

Re: Proposal: set default transaction timeout to 5 minutes

2020-05-19 Thread Anton Vinogradov
+1 On Mon, May 18, 2020 at 9:45 PM Sergey Antonov wrote: > +1 > > пн, 18 мая 2020 г. в 21:26, Andrey Mashenkov : > > > +1 > > > > On Mon, May 18, 2020 at 9:19 PM Ivan Rakov > wrote: > > > > > Hi Igniters, > > > > > > I have a very simple proposal. Let's set default TX timeout to 5 > minutes >

Re: [DISCUSS] Best way to re-encrypt existing data (TDE cache key rotation).

2020-05-14 Thread Anton Vinogradov
+1 to "In place re-encryption". - It has a simple design. - Clusters under load may require just load to re-encrypt the data. (Friendly to load). - Easy to throttle. - Easy to continue. - Design compatible with the multi-key architecture. - It can be optimized to use own WAL buffer and to

Re: Extensions for control.sh

2020-05-14 Thread Anton Vinogradov
Denis, In addition to extending the features list it's also important to find some way to allow customization of control.sh connection configuration/code. For example, it may be useful to set some attributes to binary rest client. On Thu, May 14, 2020 at 2:09 AM Denis Magda wrote: > Perfect

Re: Extensions for control.sh

2020-05-14 Thread Anton Vinogradov
such configuration to Ignite. Or am I missing something? > > Denis > > On 14.05.2020 11:43, Anton Vinogradov wrote: > > Denis, > > > > In addition to extending the features list it's also important to find > some > > way to allow customization of control.sh c

Re: [DISCUSS] Data loss handling improvements

2020-05-06 Thread Anton Vinogradov
Alexei, 1,2,4,5 - looks good to me, no objections here. >> 3. Lost state is impossible to reset if a topology doesn't have at least >> one owner for each lost partition. Do you mean that, according to your example, where >> a node2 has left, soon a node3 has left. If the node2 is returned to >>

Crash recovery speed-up #3, Cellular Switch

2020-05-06 Thread Anton Vinogradov
le.com/Non-blocking-PME-Phase-One-Node-fail-tp43531p44586.html [2] https://cwiki.apache.org/confluence/display/IGNITE/IEP-45%3A+Crash+Recovery+Speed-Up#IEP-45:CrashRecoverySpeed-Up-Cellularswitch [3] https://gist.github.com/anton-vinogradov/c50f9d0ce3e3e2997646f84ba7eba5f5#file-bench-java-L417

Re: Crash recovery speed-up #3, Cellular Switch

2020-05-12 Thread Anton Vinogradov
t are mapped into the partitions of those cells that won't be > rebalanced, is that correct? > > - > Denis > > > On Wed, May 6, 2020 at 3:24 AM Anton Vinogradov wrote: > > > Igniters, > > > > PME-free switch [1] (since 2.8) skips PME on node left when possible > &

Re: [DISCUSSION] Ignite integration testing framework.

2020-05-21 Thread Anton Vinogradov
Nikolay, Great proposal! Could you please explain how to perform testing on the different environments? For Example, you provided a rebalance test. Is it possible to perform this test with TDE enabled? Is it possible to perform it on custom OS (eg. Windows)? Is it possible to perform in on bare

Re: 2.9.1 release proposal

2020-10-20 Thread Anton Vinogradov
+1 to start the 2.9.1 release process once as soon as 2.9 released. On Mon, Oct 19, 2020 at 8:49 PM Nikolay Izhikov wrote: > Hello, Yaroslav. > > I support the idea. > But, we should carefully work with the release scope. > > IGNITE-13477 - fix for a SQL tracing that will be available only in

Re: [DISCUSSION] IEP-47 Native persistence defragmentation

2020-06-02 Thread Anton Vinogradov
Folks, Modern OS never ask you to schedule defragmentation and turn your PC off, it performs it while you're browsing. Why we're going to implement distributed system defragmentation in the old (offline) way? All you need is to implement free/reuse-list sorting. They should provide pages closest

Re: [DISCUSSION] IEP-47 Native persistence defragmentation

2020-06-02 Thread Anton Vinogradov
gmentation in the old > (offline) way?* > - because it's easier and safer, and it won't introduce any performance > degradation. > > [1] > > http://apache-ignite-developers.2346864.n4.nabble.com/How-to-free-up-space-on-disc-after-removing-entries-from-IgniteCache-with-enabled-PDS

Re: [DISCUSSION] Ignite integration testing framework.

2020-06-30 Thread Anton Vinogradov
Folks, First, I've created PR [1] with ducktests improvements PR contains the following changes - Pme-free switch proof-benchmark (2.7.6 vs master) - Ability to check (compare with) previous releases (eg. 2.7.6 & 2.8) - Global refactoring -- benchmarks javacode simplification -- services python

Re: [DISCUSSION] Ignite integration testing framework.

2020-07-09 Thread Anton Vinogradov
chTest.test.version=dev > status: PASS > run time: 1 minute 57.257 seconds > {"Streamed txs": "32924", "Measure duration (ms)": "48252", "Worst > latency (ms)": "1010"} > > -

Re: [DISCUSSION] Ignite integration testing framework.

2020-07-06 Thread Anton Vinogradov
use script [2]) and rerun the tests. [1] https://github.com/anton-vinogradov/ignite/blob/dc98ee9df90b25eb5d928090b0e78b48cae2392e/modules/ducktests/tests/docker/clean_up.sh [2] https://github.com/anton-vinogradov/ignite/blob/3c39983005bd9eaf8cb458950d942fb592fff85c/scripts/build.sh On Mon, Jul 6, 2

Re: [DISCUSSION] Ignite integration testing framework.

2020-07-02 Thread Anton Vinogradov
gt; > - 10 server nodes, preloaded with 1M of keys > > - 4 client nodes perform transactional load (client nodes physically > separated from server nodes) > > - during load: > > -- 5 server nodes stopped in parallel > > -- after 1 minute, all 5 nodes are started in parallel > >

Re: Apache Ignite 2.9.0 RELEASE [Time, Scope, Manager]

2020-06-29 Thread Anton Vinogradov
You're now at the "Ignite Release Managers" group. Please check you gain access. On Fri, Jun 26, 2020 at 9:38 PM Alex Plehanov wrote: > Guys, > > I've created the 2.9 release confluence page [1]. > Also, I found that I don't have permission to Team City release tasks. Can > anyone give me such

Re: Apache Ignite 2.9.0 RELEASE [Time, Scope, Manager]

2020-06-03 Thread Anton Vinogradov
Cellular switch On Wed, Jun 3, 2020 at 4:10 PM Nikolay Izhikov wrote: > Snapshots. > > > 3 июня 2020 г., в 16:10, Ilya Kasnacheev > написал(а): > > > > Hello! > > > > Can you please clarify what is the scope of 2.9 release? > > > > I have a feeling that we don't really have any big features in

Re: Move TcpDiscoveryZookeeperIpFinder to ignite-extensions as part of IEP-36

2020-07-24 Thread Anton Vinogradov
Ivan, Sound like an obsolete compromise partial solution kept just because of the "Zookeeper" word at its name, not because of some profit. >> So I suggest to move this single class with tests to separate module in >> ignite-extensions. +1 On Fri, Jul 24, 2020 at 11:27 AM Ivan Daschinsky

Re: [DISCUSSION] Ignite integration testing framework.

2020-07-21 Thread Anton Vinogradov
- > > > > > test_id: > > > ignitetest.tests.benchmarks.pme_free_switch_test.PmeFreeSwitchTest.test.version=2.7.6 > > > > > status: PASS > > run time: 1 minute 12.659 seconds >

Re: [VOTE] Define Apache Ignite as a Distributed Database

2020-11-24 Thread Anton Vinogradov
+1 On Tue, Nov 24, 2020 at 10:24 AM Pavel Tupitsyn wrote: > +1 > > On Tue, Nov 24, 2020 at 3:25 AM Saikat Maitra > wrote: > > > +1 > > > > On Mon, Nov 23, 2020 at 4:55 PM Valentin Kulichenko < > > valentin.kuliche...@gmail.com> wrote: > > > > > +1 > > > > > > On Mon, Nov 23, 2020 at 2:44 PM

Re: [DISCUSS] Missed (non-suited) tests

2020-11-25 Thread Anton Vinogradov
>> a > >> > TeamCity to run them? > >> > > >> > 2. Some tests marked as failed, I'll create corresponding tickets for > >> them > >> > after PR approved: > >> > - IgnitePKIndexesMigrationToUnwrapPkTest > >>

TcpDiscovery detects NONE_LEFT faster than ZookeeperDiscovery in SIGKILL case

2020-11-18 Thread Anton Vinogradov
Folks, I've found an interesting thing about delays on node failed detection depending on discovery type. For example, when we have 33 nodes cluster (33 vms in the cloud in my case), we have the following failure detection delays: ~ 10 seconds on TcpDiscovery ~ 8.5 seconds on ZookeeperDiscovery

Re: [DISCUSS] Disable socket linger by default in TCP discovery SPI.

2020-11-09 Thread Anton Vinogradov
rowse/IGNITE-13643 > > [2] https://issues.apache.org/jira/browse/IGNITE-13662 > > 05.11.2020 11:00, Anton Vinogradov пишет: > > Folks, > > Seems, we've got an agreement that the fix is necessary. > > Do we need to do except the following? > >>> zero linger as

Re: [DISCUSSION] Apache Ignite Release 2.10 (time, scope, manager)

2020-11-12 Thread Anton Vinogradov
> - Distributed environment tests [2] (Nikolay Izhikov, Anton > Vinogradov, Ivan Daschinskiy) > Will all related issues be included in the 2.10? We're at the final preparations phase. Going to start the merge discussion soon. On Wed, Nov 11, 2020 at 4:48 PM Alexey Zinoviev wrote: >

Re: [DISCUSS] Missed (non-suited) tests

2020-10-30 Thread Anton Vinogradov
Folks, What's the current state of this thread? AFAIU, we found unused and wrongly located tests and developed some checker, could we split this to some PRs? Let's merge tests usage fix and location fixes first, this will provide us an ability to automate check using Travis. On Tue, Oct 20, 2020

Re: [DISCUSS] Disable socket linger by default in TCP discovery SPI.

2020-10-30 Thread Anton Vinogradov
S 1.3, this option is unnecessary, even > if you use TLS 1.2 > > > пт, 30 окт. 2020 г. в 14:46, Anton Vinogradov : > > > Ilya > > > I think we should still keep setting linger if SSL is enabled > > Modern (updated) JVMs do not require this. > > AFAIK, P

Re: [DISCUSS] Disable socket linger by default in TCP discovery SPI.

2020-10-30 Thread Anton Vinogradov
Ilya > I think we should still keep setting linger if SSL is enabled Modern (updated) JVMs do not require this. AFAIK, Problem caused this workaround already fixed everywhere, including JDK 8. > If SSL only works with TLSv1.3 and no linger SSL works if -- TLSv1.3 with any linger -- TLSv1.2- with

Re: [DISCUSS] Ignite 3.0 development approach

2020-11-02 Thread Anton Vinogradov
Alexey, Do we have any estimates of how fast we'll be able to gain production-ready AI 3.0 in case of a "new repo" choice? On Mon, Nov 2, 2020 at 2:01 PM Alexey Goncharuk wrote: > Nikolay, > > What new features are we planning to implement for Ignite 2.x? I think once > we commence working on

Re: [DISCUSS] Missed (non-suited) tests

2020-11-02 Thread Anton Vinogradov
he new class suite. It does not look > >> pretty enough, but I think It's a path of least resistance. WDYT? > >> > >> BEFORE: > >> Module A -> SuiteA -> testA1, testA2, testB1, testB2 > >> Module B -> testB1, testB2 > >> > >> A

Re: [DISCUSS] Disable socket linger by default in TCP discovery SPI.

2020-11-05 Thread Anton Vinogradov
Folks, Seems, we've got an agreement that the fix is necessary. Do we need to do except the following? >> zero linger as default + warning on SSL enabled on JVM before the fix + warning at documentation + migration notes On Tue, Nov 3, 2020 at 2:38 PM Steshin Vladimir wrote: > Ilya, hi. >

Re: [DISCUSS] Ignite 3.0 development approach

2020-11-05 Thread Anton Vinogradov
>>>>> > > > > > >>>>>> We have bugs and issues that can be fixed in 2.x without > > > breaking > > > > > >>>> backward > > > > > >>>>> compatibility. > > > > > &

Re: [DISCUSSION] Code style. Variable abbrevations

2021-06-18 Thread Anton Vinogradov
;>> Konstantin, thanks for chiming in. > >>>>>>>> > >>>>>>>> That's exactly my concern. Overformalization is generally not a > good > >>>>>>>> thing. > >>>>>>>> Someone used "mes

Re: [DISCUSSION] Brand new distributed environment testing framework ready to be reviewed/merged.

2021-06-23 Thread Anton Vinogradov
Folks, Here's the video [1] that explains the proposal in detail. Feel free to ask questions here. [1] https://www.youtube.com/watch?v=f-i9COU5uAQ (in Russian) On Tue, Jun 8, 2021 at 2:51 PM Anton Vinogradov wrote: > Igniters, > Let me present a framework, we developed, that allows auto

[DISCUSSION] Brand new distributed environment testing framework ready to be reviewed/merged.

2021-06-08 Thread Anton Vinogradov
Igniters, Let me present a framework, we developed, that allows automating Apache Ignite testing on a real cluster. The framework was initially presented at Ignite Summit. In brief, The framework allows automating operations with any applications on a real cluster using ssh in a form of a python

Re: [DISCUSSION] Code style. Variable abbrevations

2021-06-07 Thread Anton Vinogradov
-1 here. We can fix the code and set up the rule. This will help to prevent having a weird abbreviation like "mess" (from "message") or "ign" (from "Ignite"). Also, the abbreviations list (hardcoded at IDEA plugin) allows to have same names across the whole code, this should simplify the reading.

Re: [DISCUSSION] Brand new distributed environment testing framework ready to be reviewed/merged.

2021-06-24 Thread Anton Vinogradov
Summit and wonder if you should share that recording with an > English-speaking part of the community? > > - > Denis > > On Wed, Jun 23, 2021 at 7:37 AM Anton Vinogradov wrote: > > > Folks, > > > > Here's the video [1] that explains the proposal in detail. > >

Re: [DISCUSSION] Brand new distributed environment testing framework ready to be reviewed/merged.

2021-06-28 Thread Anton Vinogradov
> -- > > Denis > > > > - > > Denis > > > > On Thu, Jun 24, 2021 at 3:43 AM Anton Vinogradov wrote: > > > >> Denis, > >> > >> Unfortunately, we had some issues with this recording (low slides > recording > >> reso

Re: [VOTE][EXTENSION] Release Apache Ignite spring-data-all-ext extensions 1.0.0 RC4

2021-06-28 Thread Anton Vinogradov
Folks, we already have Ignite release semi-automated at TeamCity. We able to assembly, publish as RC and check published. Seems, we must do the same at extensions. This way we'll check issues like this even before starting the vote. On Mon, Jun 28, 2021 at 2:14 PM Ilya Kasnacheev wrote: >

Re: [DISCUSSION] Brand new distributed environment testing framework ready to be reviewed/merged.

2021-06-28 Thread Anton Vinogradov
Seems, we have a silent agreement here :) We also discussed the proposal with the Ignite QA Meetup participants and found this contribution useful. Going to merge tomorrow, if there are no objections. On Mon, Jun 28, 2021 at 3:21 PM Anton Vinogradov wrote: > With regard to the request of I

Re: [DISCUSSION] Brand new distributed environment testing framework ready to be reviewed/merged.

2021-06-29 Thread Anton Vinogradov
-authored-by: Dmitriy Sorokin Co-authored-by: emvdovets On Mon, Jun 28, 2021 at 6:01 PM Anton Vinogradov wrote: > Seems, we have a silent agreement here :) > We also discussed the proposal with the Ignite QA Meetup participants and > found this contribution useful. > > Going to

Re: [VOTE][EXTENSION] Release Apache Ignite performance-statistics-ext extension 1.0.0 RC4

2021-06-29 Thread Anton Vinogradov
Folks, seems we MUST automate such a check, as well as the whole extension release process before the next release attempt? We may (must?) use AI release automation as a booster. Please let me know if any help required. On Tue, Jun 29, 2021 at 11:59 AM Ilya Kasnacheev wrote: > Hello! > > -1

Re: Re[2]: [DISCUSSION] Apache Ignite Release 2.10 (time, scope, manager)

2021-02-24 Thread Anton Vinogradov
Maxim, https://issues.apache.org/jira/browse/IGNITE-13873 Is ready for review, is it possible to wait for it? On Wed, Feb 24, 2021 at 12:01 AM Maxim Muzafarov wrote: > Folks, > > > I'd like to cherry-pick to the release branch the following fixes: > > Fix security context for JDBC bulk-load

Re: [VOTE] Release pyignite 0.4.0-rc1

2021-04-19 Thread Anton Vinogradov
Checked the installation from sources. +1 On Sat, Apr 17, 2021 at 7:47 PM Ivan Daschinsky wrote: > Another reason why whe should host docs on readthedocs.io only is the > fact, > that > pyignite has a separate release cycle. We will release and add more > features frequently. It's strange to

Re: [VOTE] Release pyignite 0.4.0-rc1

2021-04-19 Thread Anton Vinogradov
Also checked the hashcode generation >> from pyignite import _cutils >> print(_cutils.hashcode('test')) >> 3556498 On Mon, Apr 19, 2021 at 11:01 AM Anton Vinogradov wrote: > Checked the installation from sources. > +1 > > On Sat, Apr 17, 2021 at 7:47 PM Ivan Das

Re: Storing Teamcity projects settings in Version Control

2021-08-17 Thread Anton Vinogradov
> I think TC config should be stored in the same repo as the corresponding > code (2.x config in 2.x repo, 3.x in 3.x, etc). This will kill repo history. You'll see dozens of TC config updates vs a single Ignite fix > it would be great to be able to test them by simply creating a new branch > in

Re: [ANNOUNCE] New committer: Petr Ivanov

2021-08-19 Thread Anton Vinogradov
My regards! On Thu, Aug 19, 2021 at 1:11 PM andrei wrote: > Congrats, Petr! > > 8/19/2021 1:00 PM, Andrey Mashenkov пишет: > > Congrats, Petr! > > > > On Thu, Aug 19, 2021 at 12:58 PM Dmitry Pavlov > wrote: > > > >> Hello Ignite community, > >> > >> The Project Management Committee (PMC) for

Re: Deprecating LOCAL cache

2021-09-14 Thread Anton Vinogradov
> 1. What is expected behaviour if an old thin client requests creation of > LOCAL cache on the newest ignite cluster? Unsupported operation exception. > 2. Should we completely remove LOCAL caches support in thin clients (i.e. pyignite) before 2.13 release? Removal should happen at 2.13. On

Re: [DISCUSSION] Send documentation feedback notifications to dev list

2021-08-09 Thread Anton Vinogradov
+1 to keeping the dev list clean. On Mon, Aug 9, 2021 at 11:00 AM Pavel Tupitsyn wrote: > I agree, let's keep the dev list clean. > Automated notifications of any kind should not be sent to dev@i.a.o. > > PS Ivan, links 2 and 3 are the same. > > On Mon, Aug 9, 2021 at 8:54 AM Ivan Pavlukhin

Re: [ANNOUNCE] Welcome Zhenya Stanilovsky as a new committer

2021-07-30 Thread Anton Vinogradov
Congrats! On Fri, Jul 30, 2021 at 10:19 AM ткаленко кирилл wrote: > Zhenya, congratulations! > > Пересылаемое сообщение > 30.07.2021, 09:50, "Ivan Daschinsky" : > > > Zhenya, congrats, well deserved! > > пт, 30 июл. 2021 г. в 00:44, Andrey Mashenkov >: > > > Congratulations

Re: Secondary TeamCity instance: ci2.ignite.apache.org

2021-08-04 Thread Anton Vinogradov
Great news! Wish you good luck with this undertaking! P.s. ci2.i.a.o sounds good to me. On Tue, Aug 3, 2021 at 7:36 PM Petr Ivanov wrote: > Seems that preserving and syncing users will be difficult to achieve — > that info is stored in DB, and partial DB replication is tricky. > > Also

Re: [DISCUSSION][IEP-80] Breaking changes in Ignite-2.x

2021-10-15 Thread Anton Vinogradov
+1 On Fri, Oct 15, 2021 at 3:41 PM Nikita Amelchev wrote: > +1 for deprecation in the 2.12 release > > пт, 15 окт. 2021 г. в 15:35, Nikolay Izhikov : > > > > Hello, Igniters. > > > > I’ve prepared IEP-80 [1] to track breaking changes that should be done > in Ignite 2.x. > > > > We agreed on the

Re: NUMA aware allocator, PR review request

2021-12-06 Thread Anton Vinogradov
Let's move all GCC-related parts to ignite-extensions, release, and use them as a maven dependency. On Fri, Dec 3, 2021 at 1:08 PM Ivan Daschinsky wrote: > Ok, TC suite is ready [1]. > If there is no objections, I will merge it soon. > > Possible concerns -- now it is required to install

Re: [ANNOUNCE] Welcome Maxim Timonin as a new committer

2021-12-01 Thread Anton Vinogradov
Welcome aboard! On Wed, Dec 1, 2021 at 12:30 PM Вячеслав Коптилин wrote: > Hi, > > Congrats, Maxim! > > Thanks, > S. > > вт, 30 нояб. 2021 г. в 20:13, Maksim Timonin : > > > Hi, guys! > > > > Thanks everyone :) > > > > On Tue, Nov 30, 2021 at 1:16 PM Petr Ivanov wrote: > > > > > Great news!

Re: Re[2]: NUMA aware allocator, PR review request

2021-12-06 Thread Anton Vinogradov
Anton, I disagree. > > > >1. This should be released with main distro. > >2. This should not be abandoned. > >3. There is not any release process in ignite-extensions. > >4. Everything is working now and working good. > > > > > >So lets

Re: The conception of using two TeamCity servers

2021-12-17 Thread Anton Vinogradov
of single project. > > > > On 17 Dec 2021, at 15:14, Anton Vinogradov wrote: > > > > Petr, > > > >> I strongly suggest avoiding a separate repository for project settings. > >> Let's store them in https://github.com/apache/ignite > > > > Sou

Re: The conception of using two TeamCity servers

2021-12-17 Thread Anton Vinogradov
Petr, > I strongly suggest avoiding a separate repository for project settings. > Let's store them in https://github.com/apache/ignite Sounds good, but we must avoid dozens of additional commits in this case. Each commit should be properly formalized and related to the issue. We may create a

Re: Updating of configuration TC2: ci2.ignite.apache.org

2021-12-17 Thread Anton Vinogradov
Thanks for the update! On Thu, Dec 16, 2021 at 9:07 PM Виталий Осилов wrote: > Hi! > TeamCity server https://ci2.ignite.apache.org/ will be unavailable on > Friday 17.12 from 22:00 MSK to 00:00 MSK > Works will be done to synchronize the TeamCity configuration > https://ci2.ignite.apache.org/

Re: [CANCEL] [VOTE] Release Apache Ignite 2.11.1 RC1

2021-12-20 Thread Anton Vinogradov
ous vulnerability and it is > recommended to update log4j asap. > > пн, 20 дек. 2021 г. в 14:00, Anton Vinogradov : > > > Maxim, > > Look like an issue is not related to security problems we fix. > > Any reason to cancel the vote (delay release) to include this bugfix?

Re: [CANCEL] [VOTE] Release Apache Ignite 2.11.1 RC1

2021-12-20 Thread Anton Vinogradov
Maxim, Look like an issue is not related to security problems we fix. Any reason to cancel the vote (delay release) to include this bugfix? On Mon, Dec 20, 2021 at 1:28 PM Maxim Muzafarov wrote: > Cancelling the vote. > > I'll cherry-pick the following [1] to the release branch and prepare a >

Re: [ANNOUNCE] Welcome Vladislav Pyatkov as a new committer

2021-12-23 Thread Anton Vinogradov
Welcome aboard! On Thu, Dec 23, 2021 at 10:24 AM Ivan Pavlukhin wrote: > Congrats, Vlad! > > 2021-12-22 20:48 GMT+03:00, Ivan Daschinsky : > > Vlad, congrats! You have definitely deserved it! > > > > ср, 22 дек. 2021 г. в 20:40, Andrey Mashenkov < > andrey.mashen...@gmail.com>: > > > >>

Re: [PROPOSAL] Release Calcite-based SQL engine as an experimental feature

2021-12-30 Thread Anton Vinogradov
> it would be great to release a new SQL engine in 2.13 as an experimental feature. ++1 On Thu, Dec 30, 2021 at 12:55 PM Alex Plehanov wrote: > Andrey, > > > Is this [1] a full scope of the tickets that MUST be resolved before the > engine could be merged? > Yes, we must resolve at least these

Re: Move the Zookeeper IP-finder to the ignite-extensions

2021-12-22 Thread Anton Vinogradov
Sergei, Please make sure this will not break the Ducktests. AFAIK, we have zookeeper tests there. On Wed, Dec 22, 2021 at 12:09 PM Sergei Ryzhov wrote: > Hi, igniters, > > I've created an issue [1] to move the zookeeper IP-finder to the > ignite-extensions. The motivation is the same as with

Re: [VOTE][EXTENSION] Release Apache Ignite spring-tx-ext, spring-cache-ext extensions 1.0.0 RC1

2021-10-27 Thread Anton Vinogradov
My -0 here, since the is no automated RC preparation and testing at Apache TeamCity again. Next time, I will vote with -1 if a new vote is not covered by public automated preparation/testing as the AI release does. On Wed, Oct 27, 2021 at 11:48 AM Nikita Amelchev wrote: > Dear Ignite Community,

Re: Ci and ci2 settings synchronization

2021-11-03 Thread Anton Vinogradov
Folks, Could we have a kick-off call to determine the roadmap? On Thu, Oct 21, 2021 at 6:52 PM Denis Magda wrote: > Folks, you don't need me on that call. I'm of a little value/help here. > > - > Denis > > > On Thu, Oct 21, 2021 at 9:51 AM Nikolay Izhikov > wrote: > > > Hello, Igniters. > > >

Re: 0-day CVE in log4j

2021-12-13 Thread Anton Vinogradov
Folks, My 200 rubles here, > I want to include it to the 2.12 scope. Why not 2.11.1 as well? We should provide a fixed version for current customers asap. 2.12 require migration, while 2.11.1 can be applied as-is. On Mon, Dec 13, 2021 at 12:18 PM Stephen Darlington <

Re: [DISCUSSION] Remove outdated ignite-sys-cache

2021-12-03 Thread Anton Vinogradov
> What is the issue if third-party plugins will create «ignite-sys-cache» from the code? Great idea! On Fri, Dec 3, 2021 at 2:15 PM Nikolay Izhikov wrote: > Vyacheslav, Val, can you please clarify - What is the issue if third-party > plugins will create «ignite-sys-cache» from the code? > >

Re: [VOTE] @Nullable/@NotNull annotation usage in Ignite 3

2022-01-13 Thread Anton Vinogradov
+1 to option 4 On Thu, Jan 13, 2022 at 5:15 PM Pavel Tupitsyn wrote: > +1 to option 2 > > On Thu, Jan 13, 2022 at 3:52 PM Roman Puchkovskiy < > roman.puchkovs...@gmail.com> wrote: > > > +1 to option 2 > > > > чт, 13 янв. 2022 г. в 15:58, Sergey : > > > > > > +1 for option2 (only @Nullable) > >

Re: [VOTE] Release Apache Ignite 2.12.0 RC2

2022-01-11 Thread Anton Vinogradov
+1 On Tue, Jan 11, 2022 at 11:35 AM Maxim Muzafarov wrote: > +1 > > On Tue, 11 Jan 2022 at 11:31, Nikolay Izhikov wrote: > > > > +1 > > > > > 11 янв. 2022 г., в 02:38, Vladimir Steshin > написал(а): > > > > > > +1 > > > > > > 10.01.2022 15:52, Nikita Amelchev пишет: > > >> Dear Community, > >

<    2   3   4   5   6   7   8   9   >