[GitHub] [ignite-extensions] asfgit closed pull request #33: IGNITE-13992 Migrates Spring Transactions integration to ignite-extensions.

2021-02-08 Thread GitBox
asfgit closed pull request #33: URL: https://github.com/apache/ignite-extensions/pull/33 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [ignite] ascherbakoff commented on a change in pull request #8205: IGNITE-13393

2021-02-08 Thread GitBox
ascherbakoff commented on a change in pull request #8205: URL: https://github.com/apache/ignite/pull/8205#discussion_r572648639 ## File path: modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/atomic/GridDhtAtomicCache.java ## @@ -1038,91 +1

[GitHub] [ignite] zstan commented on a change in pull request #8759: IGNITE-14131 Fix problems with concurrent ignite.compute call

2021-02-08 Thread GitBox
zstan commented on a change in pull request #8759: URL: https://github.com/apache/ignite/pull/8759#discussion_r572630857 ## File path: modules/core/src/main/java/org/apache/ignite/spi/deployment/DeploymentSpi.java ## @@ -63,9 +63,10 @@ * Finds class loader for the given

[GitHub] [ignite] zstan commented on a change in pull request #8759: IGNITE-14131 Fix problems with concurrent ignite.compute call

2021-02-08 Thread GitBox
zstan commented on a change in pull request #8759: URL: https://github.com/apache/ignite/pull/8759#discussion_r572627141 ## File path: modules/core/src/main/java/org/apache/ignite/internal/managers/deployment/GridDeploymentLocalStore.java ## @@ -232,23 +230,33 @@ /** {@

[GitHub] [ignite] zstan commented on a change in pull request #8759: IGNITE-14131 Fix problems with concurrent ignite.compute call

2021-02-08 Thread GitBox
zstan commented on a change in pull request #8759: URL: https://github.com/apache/ignite/pull/8759#discussion_r572626169 ## File path: modules/core/src/main/java/org/apache/ignite/internal/managers/deployment/GridDeploymentLocalStore.java ## @@ -109,12 +109,10 @@ @Overrid

[GitHub] [ignite] aimiebell commented on a change in pull request #8683: IGNITE-13543 Calcite integration. Sort-based aggregates

2021-02-08 Thread GitBox
aimiebell commented on a change in pull request #8683: URL: https://github.com/apache/ignite/pull/8683#discussion_r572623007 ## File path: modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/rel/IgniteMapAggregateSort.java ## @@ -0,0 +1,142 @@ +/*

[GitHub] [ignite] aimiebell commented on a change in pull request #8711: Ignite-CDC with all fixes.

2021-02-08 Thread GitBox
aimiebell commented on a change in pull request #8711: URL: https://github.com/apache/ignite/pull/8711#discussion_r572622995 ## File path: modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheMapEntry.java ## @@ -6571,7 +6587,7 @@ private void remove

[GitHub] [ignite] ptupitsyn merged pull request #8733: IGNITE-14103 .NET: Add thin client automatic binary configuration

2021-02-08 Thread GitBox
ptupitsyn merged pull request #8733: URL: https://github.com/apache/ignite/pull/8733 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [ignite] ololo3000 edited a comment on pull request #8775: IGNITE-14142 Document control.(sh|bin) command to get an arbitrary Sy…

2021-02-08 Thread GitBox
ololo3000 edited a comment on pull request #8775: URL: https://github.com/apache/ignite/pull/8775#issuecomment-775681478 @Nikita-tech-writer Merge the master branch into your branch, please. And, please, take a look at PR [1]. It also provides documentation for system-view command (dis

[GitHub] [ignite] ololo3000 commented on pull request #8775: IGNITE-14142 Document control.(sh|bin) command to get an arbitrary Sy…

2021-02-08 Thread GitBox
ololo3000 commented on pull request #8775: URL: https://github.com/apache/ignite/pull/8775#issuecomment-775681478 @Nikita-tech-writer Merge the master branch into your branch, please. And, please, take a look at PR [1]. It also provides some documentation for system-view command (discussio

[GitHub] [ignite] Nikita-tech-writer commented on pull request #8777: IGNITE-14144 Document C++ thin client transactions

2021-02-08 Thread GitBox
Nikita-tech-writer commented on pull request #8777: URL: https://github.com/apache/ignite/pull/8777#issuecomment-775592773 @zstan Please take a look at the PR. This is an automated message from the Apache Git Service. To resp

[GitHub] [ignite] Nikita-tech-writer opened a new pull request #8777: IGNITE-14144 Document C++ thin client transactions

2021-02-08 Thread GitBox
Nikita-tech-writer opened a new pull request #8777: URL: https://github.com/apache/ignite/pull/8777 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken:

[GitHub] [ignite] Nikita-tech-writer commented on pull request #8776: IGNITE-14143 Document metric for processed keys when rebuilding indexes.

2021-02-08 Thread GitBox
Nikita-tech-writer commented on pull request #8776: URL: https://github.com/apache/ignite/pull/8776#issuecomment-775585100 @tkalkirill Please take a look at the PR. This is an automated message from the Apache Git Service. To

[GitHub] [ignite] Nikita-tech-writer opened a new pull request #8776: IGNITE-14143 Document metric for processed keys when rebuilding indexes.

2021-02-08 Thread GitBox
Nikita-tech-writer opened a new pull request #8776: URL: https://github.com/apache/ignite/pull/8776 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken:

[GitHub] [ignite] Nikita-tech-writer commented on pull request #8775: IGNITE-14142 Document control.(sh|bin) command to get an arbitrary Sy…

2021-02-08 Thread GitBox
Nikita-tech-writer commented on pull request #8775: URL: https://github.com/apache/ignite/pull/8775#issuecomment-775567764 @ololo3000 Please have a look a the PR. This is an automated message from the Apache Git Service. To r

[GitHub] [ignite] Nikita-tech-writer opened a new pull request #8775: IGNITE-14142 Document control.(sh|bin) command to get an arbitrary Sy…

2021-02-08 Thread GitBox
Nikita-tech-writer opened a new pull request #8775: URL: https://github.com/apache/ignite/pull/8775 …stemView Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have

[GitHub] [ignite] Nikita-tech-writer commented on pull request #8703: Ignite 13385 3

2021-02-08 Thread GitBox
Nikita-tech-writer commented on pull request #8703: URL: https://github.com/apache/ignite/pull/8703#issuecomment-775537866 @dmagda Please see the changes to this PR. This is an automated message from the Apache Git Service. T

[GitHub] [ignite] Nikita-tech-writer commented on a change in pull request #8703: Ignite 13385 3

2021-02-08 Thread GitBox
Nikita-tech-writer commented on a change in pull request #8703: URL: https://github.com/apache/ignite/pull/8703#discussion_r572453895 ## File path: docs/_docs/memory-configuration/data-regions.adoc ## @@ -82,3 +82,72 @@ include::code-snippets/dotnet/MemoryArchitecture.cs[tag=m

[GitHub] [ignite] Nikita-tech-writer commented on pull request #8756: IGNITE-14129 Add documentation for .NET thin client service invocation

2021-02-08 Thread GitBox
Nikita-tech-writer commented on pull request #8756: URL: https://github.com/apache/ignite/pull/8756#issuecomment-775505926 @Mmuzaf, I believe that now we can merge the changes to the master and to the Ignite 2.10. This is an

[GitHub] [ignite] Mmuzaf opened a new pull request #8774: IGNITE-14141 Remove unnecessary storage configuration from PageStore

2021-02-08 Thread GitBox
Mmuzaf opened a new pull request #8774: URL: https://github.com/apache/ignite/pull/8774 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### The Co

[GitHub] [ignite] vldpyatkov commented on a change in pull request #8759: IGNITE-14131 Fix problems with concurrent ignite.compute call

2021-02-08 Thread GitBox
vldpyatkov commented on a change in pull request #8759: URL: https://github.com/apache/ignite/pull/8759#discussion_r572242933 ## File path: modules/core/src/main/java/org/apache/ignite/internal/managers/deployment/GridDeploymentLocalStore.java ## @@ -232,23 +230,33 @@ /

[GitHub] [ignite] ptupitsyn opened a new pull request #8773: IGNITE-14116 .NET: Update tests with LongRunning category

2021-02-08 Thread GitBox
ptupitsyn opened a new pull request #8773: URL: https://github.com/apache/ignite/pull/8773 * Review all tests duration and mark slow ones with `[Category(TestUtils.CategoryIntensive)]` * Update DEVNOTES with a command to run quick tests only | | All Tests | Quick Tests |

[GitHub] [ignite] ptupitsyn commented on a change in pull request #8756: IGNITE-14129 Add documentation for .NET thin client service invocation

2021-02-08 Thread GitBox
ptupitsyn commented on a change in pull request #8756: URL: https://github.com/apache/ignite/pull/8756#discussion_r572226639 ## File path: docs/_docs/thin-clients/dotnet-thin-client.adoc ## @@ -235,6 +235,20 @@ task named `org.foo.bar.AddOneTask` passing `1` as an input parame

[GitHub] [ignite] ptupitsyn commented on a change in pull request #8756: IGNITE-14129 Add documentation for .NET thin client service invocation

2021-02-08 Thread GitBox
ptupitsyn commented on a change in pull request #8756: URL: https://github.com/apache/ignite/pull/8756#discussion_r572225999 ## File path: docs/_docs/thin-clients/dotnet-thin-client.adoc ## @@ -235,6 +235,20 @@ task named `org.foo.bar.AddOneTask` passing `1` as an input parame

[GitHub] [ignite] ptupitsyn merged pull request #8760: IGNITE-10073 .NET: Document how to exclude jar files from build output

2021-02-08 Thread GitBox
ptupitsyn merged pull request #8760: URL: https://github.com/apache/ignite/pull/8760 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [ignite] vldpyatkov opened a new pull request #8772: IGNITE-14140 Checkpointer thread holds write lock too long

2021-02-08 Thread GitBox
vldpyatkov opened a new pull request #8772: URL: https://github.com/apache/ignite/pull/8772 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### Th

[GitHub] [ignite] Vladsz83 opened a new pull request #8771: Fix ducktape v2

2021-02-08 Thread GitBox
Vladsz83 opened a new pull request #8771: URL: https://github.com/apache/ignite/pull/8771 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [ignite] AMashenkov commented on a change in pull request #8490: IGNITE-13056 Move indexes to core

2021-02-08 Thread GitBox
AMashenkov commented on a change in pull request #8490: URL: https://github.com/apache/ignite/pull/8490#discussion_r572034957 ## File path: modules/control-utility/src/test/java/org/apache/ignite/util/GridCommandHandlerIndexForceRebuildTest.java ## @@ -558,11 +558,11 @@ privat

[GitHub] [ignite] timoninmaxim edited a comment on pull request #8490: IGNITE-13056 Move indexes to core

2021-02-08 Thread GitBox
timoninmaxim edited a comment on pull request #8490: URL: https://github.com/apache/ignite/pull/8490#issuecomment-775201344 Hi @korlov42 ! Thanks a lot for reviewing my PR! You ask right questions. I worked on this PR pretty much time and then need to restore why I did some things this way

[GitHub] [ignite] vldpyatkov opened a new pull request #8770: IGNITE-14139 Incorrect initialize checkpoint-runner-cpu thread pool

2021-02-08 Thread GitBox
vldpyatkov opened a new pull request #8770: URL: https://github.com/apache/ignite/pull/8770 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### Th

[GitHub] [ignite] timoninmaxim edited a comment on pull request #8490: IGNITE-13056 Move indexes to core

2021-02-08 Thread GitBox
timoninmaxim edited a comment on pull request #8490: URL: https://github.com/apache/ignite/pull/8490#issuecomment-775201344 Hi @korlov42 ! Thanks a lot for reviewing my PR! You ask right questions. I worked on this PR pretty much time and then need to restore why I did some things this way

[GitHub] [ignite] timoninmaxim commented on pull request #8490: IGNITE-13056 Move indexes to core

2021-02-08 Thread GitBox
timoninmaxim commented on pull request #8490: URL: https://github.com/apache/ignite/pull/8490#issuecomment-775201344 Hi @korlov42 ! Thanks a lot for reviewing my PR! You ask right questions. I worked on this PR pretty much time and then need to restore why I did some things this way. So I'

[GitHub] [ignite] asfgit closed pull request #8743: IGNITE-14112 Revisit usages of GridClosureProcessor.runLocalSafe and GridClosureProcessor.callLocalSafe methods

2021-02-08 Thread GitBox
asfgit closed pull request #8743: URL: https://github.com/apache/ignite/pull/8743 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [ignite-python-thin-client] asfgit closed pull request #10: IGNITE-13967 Optimize and refactor of parsing response from server.

2021-02-08 Thread GitBox
asfgit closed pull request #10: URL: https://github.com/apache/ignite-python-thin-client/pull/10 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [ignite] Nikita-tech-writer commented on a change in pull request #8760: IGNITE-10073 .NET: Document how to exclude jar files from build output

2021-02-08 Thread GitBox
Nikita-tech-writer commented on a change in pull request #8760: URL: https://github.com/apache/ignite/pull/8760#discussion_r572049287 ## File path: docs/_docs/net-specific/net-deployment-options.adoc ## @@ -17,35 +17,47 @@ == Overview Apache Ignite.NET consists of .NET asse

[GitHub] [ignite] Nikita-tech-writer commented on a change in pull request #8760: IGNITE-10073 .NET: Document how to exclude jar files from build output

2021-02-08 Thread GitBox
Nikita-tech-writer commented on a change in pull request #8760: URL: https://github.com/apache/ignite/pull/8760#discussion_r572048954 ## File path: docs/_docs/net-specific/net-deployment-options.adoc ## @@ -17,35 +17,47 @@ == Overview Apache Ignite.NET consists of .NET asse

[GitHub] [ignite] Nikita-tech-writer commented on a change in pull request #8760: IGNITE-10073 .NET: Document how to exclude jar files from build output

2021-02-08 Thread GitBox
Nikita-tech-writer commented on a change in pull request #8760: URL: https://github.com/apache/ignite/pull/8760#discussion_r572048456 ## File path: docs/_docs/net-specific/net-deployment-options.adoc ## @@ -17,35 +17,47 @@ == Overview Apache Ignite.NET consists of .NET asse

[GitHub] [ignite-python-thin-client] ivandasch commented on a change in pull request #10: IGNITE-13967 Optimize and refactor of parsing response from server.

2021-02-08 Thread GitBox
ivandasch commented on a change in pull request #10: URL: https://github.com/apache/ignite-python-thin-client/pull/10#discussion_r572043411 ## File path: pyignite/connection/__init__.py ## @@ -97,7 +97,7 @@ def _check_ssl_params(params): ).format(param))

[GitHub] [ignite-python-thin-client] ivandasch commented on a change in pull request #10: IGNITE-13967 Optimize and refactor of parsing response from server.

2021-02-08 Thread GitBox
ivandasch commented on a change in pull request #10: URL: https://github.com/apache/ignite-python-thin-client/pull/10#discussion_r572043042 ## File path: pyignite/stream/binary_stream.py ## @@ -0,0 +1,114 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] [ignite-python-thin-client] ivandasch commented on a change in pull request #10: IGNITE-13967 Optimize and refactor of parsing response from server.

2021-02-08 Thread GitBox
ivandasch commented on a change in pull request #10: URL: https://github.com/apache/ignite-python-thin-client/pull/10#discussion_r572042192 ## File path: pyignite/stream/binary_stream.py ## @@ -0,0 +1,114 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] [ignite] vldpyatkov opened a new pull request #8769: IGNITE-14138 Historical rebalance kills cluster

2021-02-08 Thread GitBox
vldpyatkov opened a new pull request #8769: URL: https://github.com/apache/ignite/pull/8769 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### Th

[GitHub] [ignite-python-thin-client] isapego commented on a change in pull request #10: IGNITE-13967 Optimize and refactor of parsing response from server.

2021-02-08 Thread GitBox
isapego commented on a change in pull request #10: URL: https://github.com/apache/ignite-python-thin-client/pull/10#discussion_r571976014 ## File path: pyignite/connection/__init__.py ## @@ -97,7 +97,7 @@ def _check_ssl_params(params): ).format(param))

[GitHub] [ignite] Vladsz83 opened a new pull request #8768: Fix ducktape tests

2021-02-08 Thread GitBox
Vladsz83 opened a new pull request #8768: URL: https://github.com/apache/ignite/pull/8768 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [ignite] timoninmaxim commented on a change in pull request #8686: Password based authentication support in ducktape tests

2021-02-08 Thread GitBox
timoninmaxim commented on a change in pull request #8686: URL: https://github.com/apache/ignite/pull/8686#discussion_r572009018 ## File path: modules/ducktests/tests/ignitetest/services/utils/control_utility.py ## @@ -313,6 +303,39 @@ def __parse_output(raw_output): def __

[GitHub] [ignite] Vladsz83 closed pull request #8747: Fix ducktape failures

2021-02-08 Thread GitBox
Vladsz83 closed pull request #8747: URL: https://github.com/apache/ignite/pull/8747 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [ignite] Mmuzaf opened a new pull request #8767: IGNITE-14134 commit snapshot partition iterator

2021-02-08 Thread GitBox
Mmuzaf opened a new pull request #8767: URL: https://github.com/apache/ignite/pull/8767 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### The Co

[GitHub] [ignite] anton-vinogradov merged pull request #8522: GNITE-13835 : Improve discovery ducktape test to research small timeouts and behavior on large cluster.

2021-02-08 Thread GitBox
anton-vinogradov merged pull request #8522: URL: https://github.com/apache/ignite/pull/8522 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [ignite] alex-plekhanov closed pull request #8766: IGNITE-14136 Fix flaky ServicesTest.testServiceTimeout - 500 TC runs

2021-02-08 Thread GitBox
alex-plekhanov closed pull request #8766: URL: https://github.com/apache/ignite/pull/8766 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [ignite] alex-plekhanov commented on a change in pull request #8733: IGNITE-14103 .NET: Add thin client automatic binary configuration

2021-02-08 Thread GitBox
alex-plekhanov commented on a change in pull request #8733: URL: https://github.com/apache/ignite/pull/8733#discussion_r571870033 ## File path: modules/core/src/main/java/org/apache/ignite/internal/processors/platform/client/binary/ClientBinaryConfigurationGetRequest.java ## @

[GitHub] [ignite] ibessonov merged pull request #8698: IGNITE-14055 Execution of timeout handshake object moved to the separ…

2021-02-08 Thread GitBox
ibessonov merged pull request #8698: URL: https://github.com/apache/ignite/pull/8698 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [ignite] alex-plekhanov opened a new pull request #8766: IGNITE-14136 Fix flaky ServicesTest.testServiceTimeout - 500 TC runs

2021-02-08 Thread GitBox
alex-plekhanov opened a new pull request #8766: URL: https://github.com/apache/ignite/pull/8766 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ##

[GitHub] [ignite] alex-plekhanov opened a new pull request #8765: IGNITE-14136 Fix flaky ServicesTest.testServiceTimeout

2021-02-08 Thread GitBox
alex-plekhanov opened a new pull request #8765: URL: https://github.com/apache/ignite/pull/8765 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ##