[GitHub] ignite pull request #5575: IGNITE-10535

2018-12-05 Thread devozerov
GitHub user devozerov opened a pull request: https://github.com/apache/ignite/pull/5575 IGNITE-10535 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-10535-1 Alternatively you can review and apply th

[jira] [Created] (IGNITE-10535) SQL: extract partition mapping logic from GridReduceQueryExecutor into a separate class

2018-12-05 Thread Vladimir Ozerov (JIRA)
Vladimir Ozerov created IGNITE-10535: Summary: SQL: extract partition mapping logic from GridReduceQueryExecutor into a separate class Key: IGNITE-10535 URL: https://issues.apache.org/jira/browse/IGNITE-10535

[GitHub] ignite pull request #3451: IGNITE-7366 Service reassignment with merge excha...

2018-12-05 Thread xtern
Github user xtern closed the pull request at: https://github.com/apache/ignite/pull/3451 ---

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitrii Ryabov
Anton, I think wrapping every disconnecting node with try-catch will be less readable than no-op handler. ср, 5 дек. 2018 г., 9:26 Dmitriy Pavlov dpav...@apache.org: > Folks let me remind you that Dmitry changed default of ALL tests from noop > to a meaningful handler. So we should start every me

[GitHub] ignite pull request #1074: Ignite 3810

2018-12-05 Thread dgovorukhin
Github user dgovorukhin closed the pull request at: https://github.com/apache/ignite/pull/1074 ---

[jira] [Created] (IGNITE-10537) MVCC: Client reconnect tests became unstable after mvcc coordinator reassign fix.

2018-12-05 Thread Roman Kondakov (JIRA)
Roman Kondakov created IGNITE-10537: --- Summary: MVCC: Client reconnect tests became unstable after mvcc coordinator reassign fix. Key: IGNITE-10537 URL: https://issues.apache.org/jira/browse/IGNITE-10537

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Dmitrii, No-op means "hide any problem", so, we lose the guarantees. Could you please share some examples where "no-op" better than "strict try-catch with a check"? On Wed, Dec 5, 2018 at 11:37 AM Dmitrii Ryabov wrote: > Anton, I think wrapping every disconnecting node with try-catch will be >

[jira] [Created] (IGNITE-10536) Incorrect hanlding of error in agent manager

2018-12-05 Thread Alexander Kalinin (JIRA)
Alexander Kalinin created IGNITE-10536: -- Summary: Incorrect hanlding of error in agent manager Key: IGNITE-10536 URL: https://issues.apache.org/jira/browse/IGNITE-10536 Project: Ignite I

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
Anton, If I understood this idea right, try-catch will not work because failure can be thrown into an Ignite thread pool, which catches any exceptions and errors. Which code block will do a throw? Sincerely, Dmitriy Pavlov ср, 5 дек. 2018 г. в 12:16, Anton Vinogradov : > Dmitrii, > > No-op me

Re: welcome letter

2018-12-05 Thread Dmitriy Pavlov
Hi Srinivas, You are in! Welcome aboard. Sincerely, Dmitriy Pavlov ср, 5 дек. 2018 г. в 06:03, Srinivas Reddy : > Can you please add me to contributors group? > > Jira : mrsrinivas > > Thank you > > - > Srinivas > > - Typed on tiny keys. pls ignore typos.{mobile app} > > On Tue, 4 Dec, 2018, 04

Re: Default failure handler was changed for tests

2018-12-05 Thread Andrey Mashenkov
Hi, Dmitri, The meaningful failure handler as a default one looks reasonable. Thanks a lot. But what is the reason to fallback to noop for 100+ test? Does it means these test become failed after changing default failure handler? If so, let's create a ticket (may be umbrella) to investigate and f

[jira] [Created] (IGNITE-10538) Web console: Implement common component for empty ui-grid messages

2018-12-05 Thread Vasiliy Sisko (JIRA)
Vasiliy Sisko created IGNITE-10538: -- Summary: Web console: Implement common component for empty ui-grid messages Key: IGNITE-10538 URL: https://issues.apache.org/jira/browse/IGNITE-10538 Project: Ign

Re: [Result][VOTE] Creation dedicated list for github notifiacations

2018-12-05 Thread Eduard Shangareev
https://issues.apache.org/jira/browse/INFRA-17351 A ticket was created. On Fri, Nov 30, 2018 at 12:04 AM Denis Magda wrote: > A request has been submitted. > > -- > Denis > > On Thu, Nov 29, 2018 at 11:45 AM Dmitriy Pavlov > wrote: > > > Denis, could you please create a new list for Apache Igni

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Dmitriy, >> Which code block will do a throw? Depends on the test. Looks like we make the *bad *test even *worse*. That's not a correct fix. In case you expect failure you have to check this expectation inside the special handler. I'd like to ask you to rollback these changes and replace them w

[jira] [Created] (IGNITE-10539) [ML] Make 'with' methods consistent

2018-12-05 Thread Artem Malykh (JIRA)
Artem Malykh created IGNITE-10539: - Summary: [ML] Make 'with' methods consistent Key: IGNITE-10539 URL: https://issues.apache.org/jira/browse/IGNITE-10539 Project: Ignite Issue Type: Improvem

[GitHub] ignite pull request #5559: IGNITE-10380: Drop Multi-label Classification for...

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/5559 ---

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
I will not do any rollback because changes make tests better. Please pay attention that no-op became default long time ago. Please discuss this selection with authors of the previous commit. New commit changes NoOp->FailTest+stopNode. Please provide a PR to demonstrate your idea how to transfer an

[GitHub] ignite pull request #5472: IGNITE-9023 LinkageError or ClassNotFoundExceptio...

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/5472 ---

Re: [jira] [Created] (IGNITE-10539) [ML] Make 'with' methods consistent

2018-12-05 Thread Alexey Zinoviev
In my opinion, the main senario is the 'in-place' changes via with method. This is a Java and mutability is normal behaviour. ср, 5 дек. 2018 г. в 13:05, Artem Malykh (JIRA) : > Artem Malykh created IGNITE-10539: > - > > Summary: [ML] Make 'with' m

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Dmitriy, As I said before, these changes allow tests to be successful in case of unexpected failures. That's not acceptable. As a reviewer, you have to be ready to provide arguments why these tests have to be fixed this way and what was the problem, in case you merged such changes. That's unaccep

[jira] [Created] (IGNITE-10540) [TC Bot] "Compare builds" page with Internal Server Error [500]

2018-12-05 Thread Nikolai Kulagin (JIRA)
Nikolai Kulagin created IGNITE-10540: Summary: [TC Bot] "Compare builds" page with Internal Server Error [500] Key: IGNITE-10540 URL: https://issues.apache.org/jira/browse/IGNITE-10540 Project: Ig

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
Anton, please provide PR to demo your idea. Code speaks louder than words sometimes. No reason to revert a contribution if someone has an idea, which is not clear for others. Again, we should discuss not Dmitrii contribution, but the initial selection of no-op. If you will do a test failure fixe

[GitHub] zzzadruga opened a new pull request #87: IGNITE-10540 "Compare builds" page with Internal Server Error [500]

2018-12-05 Thread GitBox
zzzadruga opened a new pull request #87: IGNITE-10540 "Compare builds" page with Internal Server Error [500] URL: https://github.com/apache/ignite-teamcity-bot/pull/87 This is an automated message from the Apache Git Service

[GitHub] SomeFire commented on a change in pull request #87: IGNITE-10540 "Compare builds" page with Internal Server Error [500]

2018-12-05 Thread GitBox
SomeFire commented on a change in pull request #87: IGNITE-10540 "Compare builds" page with Internal Server Error [500] URL: https://github.com/apache/ignite-teamcity-bot/pull/87#discussion_r239018971 ## File path: ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/build

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Dmitriy, That's an incorrect idea to ask me to provide PR or to fix these test properly since I'm not an author or reviewer. But, I, as a community member, ask you to explain what problems the fix fixes. In case you're not able to provide the explanation I will rollback the changes. That's not a

[jira] [Created] (IGNITE-10542) [ML] Distributive models with basic algebra

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10542: --- Summary: [ML] Distributive models with basic algebra Key: IGNITE-10542 URL: https://issues.apache.org/jira/browse/IGNITE-10542 Project: Ignite Issue Type: New

[jira] [Created] (IGNITE-10543) [ML] Test/train sample generator

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10543: --- Summary: [ML] Test/train sample generator Key: IGNITE-10543 URL: https://issues.apache.org/jira/browse/IGNITE-10543 Project: Ignite Issue Type: New Feature

[GitHub] ololo3000 opened a new pull request #88: IGNITE-10519 JavaDocs added

2018-12-05 Thread GitBox
ololo3000 opened a new pull request #88: IGNITE-10519 JavaDocs added URL: https://github.com/apache/ignite-teamcity-bot/pull/88 This is an automated message from the Apache Git Service. To respond to the message, please log o

[jira] [Created] (IGNITE-10544) [ML] GMM with fixed components

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10544: --- Summary: [ML] GMM with fixed components Key: IGNITE-10544 URL: https://issues.apache.org/jira/browse/IGNITE-10544 Project: Ignite Issue Type: New Feature

[jira] [Created] (IGNITE-10545) [ML] Kullback–Leibler divergence

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10545: --- Summary: [ML] Kullback–Leibler divergence Key: IGNITE-10545 URL: https://issues.apache.org/jira/browse/IGNITE-10545 Project: Ignite Issue Type: New Feature

[jira] [Created] (IGNITE-10547) [ML] Examples of GMM usage

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10547: --- Summary: [ML] Examples of GMM usage Key: IGNITE-10547 URL: https://issues.apache.org/jira/browse/IGNITE-10547 Project: Ignite Issue Type: New Feature

[jira] [Created] (IGNITE-10541) [ML] Umbrella: EM (GMM) with adding and removal of components

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10541: --- Summary: [ML] Umbrella: EM (GMM) with adding and removal of components Key: IGNITE-10541 URL: https://issues.apache.org/jira/browse/IGNITE-10541 Project: Ignite

[jira] [Created] (IGNITE-10548) [ML] Classificator based on GMM

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10548: --- Summary: [ML] Classificator based on GMM Key: IGNITE-10548 URL: https://issues.apache.org/jira/browse/IGNITE-10548 Project: Ignite Issue Type: New Feature

[jira] [Created] (IGNITE-10546) [ML] GMM with adding and removal of components

2018-12-05 Thread Yury Babak (JIRA)
Yury Babak created IGNITE-10546: --- Summary: [ML] GMM with adding and removal of components Key: IGNITE-10546 URL: https://issues.apache.org/jira/browse/IGNITE-10546 Project: Ignite Issue Type: N

[GitHub] ignite pull request #5576: IGNITE-10507 idle_verify added CRC sum check and ...

2018-12-05 Thread antonovsergey93
GitHub user antonovsergey93 opened a pull request: https://github.com/apache/ignite/pull/5576 IGNITE-10507 idle_verify added CRC sum check and collecting exceptions from all nodes You can merge this pull request into a Git repository by running: $ git pull https://github.com/

Re: Default failure handler was changed for tests

2018-12-05 Thread Павлухин Иван
Hi Anton, Could you please summarize what does aforementioned patch made really worse? As I see, the patch added a very good thing -- meaningful failure handler in tests. And I think it is really important. But was is the harm and does it overweight positive result? And why? ср, 5 дек. 2018 г. в

[jira] [Created] (IGNITE-10549) [ML] LogisticRegressionSGDTrainerTest fails with BLAS

2018-12-05 Thread Alexey Platonov (JIRA)
Alexey Platonov created IGNITE-10549: Summary: [ML] LogisticRegressionSGDTrainerTest fails with BLAS Key: IGNITE-10549 URL: https://issues.apache.org/jira/browse/IGNITE-10549 Project: Ignite

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Ivan, Do you mean massive no-op handler restore patch [1]? [1] https://github.com/apache/ignite/pull/4974/files On Wed, Dec 5, 2018 at 2:53 PM Павлухин Иван wrote: > Hi Anton, > > Could you please summarize what does aforementioned patch made really > worse? > > As I see, the patch added a ve

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
Dmitrii Ryabov explained these tests are perfectly ok to have failures as these tests do test failures. Anton, there is no reason to revert other's contributions because you know how to do things better. A lot of people can do things better than me. Should we revert everything I've contributed? I

[jira] [Created] (IGNITE-10550) MVCC: rework IgniteWalFlush* tests for mvcc.

2018-12-05 Thread Andrew Mashenkov (JIRA)
Andrew Mashenkov created IGNITE-10550: - Summary: MVCC: rework IgniteWalFlush* tests for mvcc. Key: IGNITE-10550 URL: https://issues.apache.org/jira/browse/IGNITE-10550 Project: Ignite Iss

Re: Default failure handler was changed for tests

2018-12-05 Thread Павлухин Иван
Anton, Yes I meant that patch. And I would like to respell a name "massive no-op handler restore" to "use no-op failure handler only where it is assumed". ср, 5 дек. 2018 г. в 15:09, Dmitriy Pavlov : > > Dmitrii Ryabov explained these tests are perfectly ok to have failures as > these tests do tes

[GitHub] ignite pull request #5577: IGNITE-10549: LogisticRegressionSGDTrainerTest fa...

2018-12-05 Thread avplatonov
GitHub user avplatonov opened a pull request: https://github.com/apache/ignite/pull/5577 IGNITE-10549: LogisticRegressionSGDTrainerTest fails with BLAS You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite

[jira] [Created] (IGNITE-10551) MVCC: IgniteWalHistoryReservations tests failed.

2018-12-05 Thread Andrew Mashenkov (JIRA)
Andrew Mashenkov created IGNITE-10551: - Summary: MVCC: IgniteWalHistoryReservations tests failed. Key: IGNITE-10551 URL: https://issues.apache.org/jira/browse/IGNITE-10551 Project: Ignite

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Dmitriy Pavlov, Dmitrii Ryabov, >> Anton, there is no reason to revert other's contributions because you know >> how to do things better. What I see is "We replaced no-op with the proper handler, but . 100+ no-op still here because tests start failing :)" That's a completely different situati

[GitHub] asfgit closed pull request #87: IGNITE-10540 "Compare builds" page with Internal Server Error [500]

2018-12-05 Thread GitBox
asfgit closed pull request #87: IGNITE-10540 "Compare builds" page with Internal Server Error [500] URL: https://github.com/apache/ignite-teamcity-bot/pull/87 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of pr

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Ivan, >> Yes I meant that patch. And I would like to respell a name "massive >> no-op handler restore" to "use no-op failure handler only where it is >> assumed". How about "we changed some handlers to proper, but keep other no-ops using explicit copy-paste"? :) On Wed, Dec 5, 2018 at 3:38 PM An

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
Anton, I disagree with this approach: "You will ask, other will provide explanations/excuses/apology and so on". Since you rejecting to chime in and help this means trying to manage instead of doing. I don't ask you to re-do this change, I ask to demonstrate any better approach for tests which int

Re: Default failure handler was changed for tests

2018-12-05 Thread Eduard Shangareev
Guys, I didn't get. What is the problem in saving No-Op for several tests? Why should we keep No-Op for all? On Wed, Dec 5, 2018 at 3:20 PM Павлухин Иван wrote: > Anton, > > Yes I meant that patch. And I would like to respell a name "massive > no-op handler restore" to "use no-op failure handle

[GitHub] ignite pull request #5578: IGNITE_DISABLE_WAL_DURING_REBALANCING turned on b...

2018-12-05 Thread sergey-chugunov-1985
GitHub user sergey-chugunov-1985 opened a pull request: https://github.com/apache/ignite/pull/5578 IGNITE_DISABLE_WAL_DURING_REBALANCING turned on by default, test for race between checkpointer and affinity change added You can merge this pull request into a Git repository by runn

[GitHub] ignite pull request #5499: IGNITE-10049: Cache 4 test suite for MVCC mode.

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/5499 ---

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
>> I didn't get. What is the problem in saving No-Op for several tests? Why >> should we keep No-Op for all? Several (less than 10) is ok to me with the proper explanation why tests fail and why no-op is a better choice. 100+++ copy-pasted no-op handlers are not ok! >> I don't ask you to re-do th

[GitHub] ignite pull request #5579: IGNITE-9322: MVCC deadlock detection

2018-12-05 Thread pavlukhin
GitHub user pavlukhin opened a pull request: https://github.com/apache/ignite/pull/5579 IGNITE-9322: MVCC deadlock detection You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-9322 Alternatively you ca

[GitHub] ignite pull request #5549: IGNITE-10410: Create "Cache 7" test suite for MVC...

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/5549 ---

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
Anton, if you care enough here will you try to research a couple of these tests? Or you are asking others to do things for you, aren't you? I like idea from Andrew to create ticket and check these test to keep moving towards 010 tests with noop. It is easy to locate these overridden method now

[GitHub] ignite pull request #5556: IGNITE-10368: Add mvcc cache test suite 9.

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/5556 ---

[GitHub] ignite pull request #5580: IGNITE-9303 : Fixed flag check on page recycling ...

2018-12-05 Thread ilantukh
GitHub user ilantukh opened a pull request: https://github.com/apache/ignite/pull/5580 IGNITE-9303 : Fixed flag check on page recycling and changed PageMemoryTracker to use forced read lock. You can merge this pull request into a Git repository by running: $ git pull https://

[GitHub] ignite pull request #5555: IGNITE-10509 reordered setting of timeout flag an...

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/ ---

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Dmitriy, It's ok in case someone ready to do this (get rid of all no-op or explain why it's a better choice). Explicit confirmation required. Otherwise, only rollback is an option. On Wed, Dec 5, 2018 at 4:29 PM Dmitriy Pavlov wrote: > Anton, if you care enough here will you try to research a

Re: Default failure handler was changed for tests

2018-12-05 Thread Andrey Kuznetsov
Anton, I really like your perfectionism. But why should we make all things perfect in a single fix? The change you want to roll back is definitely useful for the project: the majority of our tests do not hide potential bugs under no-op handler anymore, and the small number of tests require additio

Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Vyacheslav Daradur
Igniters, I've found that the project's test framework uses 'TcpDiscoveryMulticastIpFinder' as default IP finder for tests and there are a lot of tests written by Ignite's experts that override it to 'TcpDiscoveryVmIpFinder'. Most of our tests starting Ignite nodes in the same JVM, that allows us

Re: Default failure handler was changed for tests

2018-12-05 Thread Andrey Mashenkov
Dmitry, Do we have TC run results for the PR before massive failure handler fallbacks were added? Let's create a ticket to investigate possibility of using any meaningful failure handler for such tests with TC report attached. On Wed, Dec 5, 2018 at 4:41 PM Anton Vinogradov wrote: > Dmitriy, >

Re: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Vladimir Ozerov
Huge +1 On Wed, Dec 5, 2018 at 5:09 PM Vyacheslav Daradur wrote: > Igniters, > > I've found that the project's test framework uses > 'TcpDiscoveryMulticastIpFinder' as default IP finder for tests and > there are a lot of tests written by Ignite's experts that override it > to 'TcpDiscoveryVmIpFi

[GitHub] SomeFire opened a new pull request #89: IGNITE-10454 Create page with muted tests

2018-12-05 Thread GitBox
SomeFire opened a new pull request #89: IGNITE-10454 Create page with muted tests URL: https://github.com/apache/ignite-teamcity-bot/pull/89 This is an automated message from the Apache Git Service. To respond to the message

Re: Default failure handler was changed for tests

2018-12-05 Thread Anton Vinogradov
Andrey, >> But why should we make all things perfect >> in a single fix? As I said, I'm ok in case someone ready to continue :) But, we should avoid such over-copy-pasted commits in the future. On Wed, Dec 5, 2018 at 5:13 PM Andrey Mashenkov wrote: > Dmitry, > > Do we have TC run results for th

Re: Default failure handler was changed for tests

2018-12-05 Thread Nikolay Izhikov
Hello, Igniters. I'm agree with Anton Vinogradov. I think we should avoid commits like [1] Copy paste coding style is well known anti pattern. Don't we have another option to do same fix with better styling? Accepting such patches leads to the further tickets to cleanup mess that patches brings

Re: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Denis Mekhanikov
Slava, These are exactly my thoughts, so I fully support you here. I already wrote about it: http://apache-ignite-developers.2346864.n4.nabble.com/IP-finder-in-tests-td33322.html But I kind of abandoned this activity. Feel free to take over it. Denis ср, 5 дек. 2018 г. в 17:22, Vladimir Ozerov :

[jira] [Created] (IGNITE-10552) Web Agent: Improve logging when cluster topology changed

2018-12-05 Thread Alexey Kuznetsov (JIRA)
Alexey Kuznetsov created IGNITE-10552: - Summary: Web Agent: Improve logging when cluster topology changed Key: IGNITE-10552 URL: https://issues.apache.org/jira/browse/IGNITE-10552 Project: Ignite

[GitHub] ignite pull request #5581: IGNITE-10552 Web Agent: Improve logging when clus...

2018-12-05 Thread akuznetsov-gridgain
GitHub user akuznetsov-gridgain opened a pull request: https://github.com/apache/ignite/pull/5581 IGNITE-10552 Web Agent: Improve logging when cluster topology changed You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apac

Re: Default failure handler was changed for tests

2018-12-05 Thread Andrey Kuznetsov
As I can see from the above discussion, > Tests in these classes check fail cases when we expect critical failure like node stop or exception thrown So, this copy-n-paste-style change is caused by the imperfect logic of existing tests, that should be reworked in more robust way, e.g. using custo

Re: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Anton Vinogradov
++1 On Wed, Dec 5, 2018 at 5:55 PM Denis Mekhanikov wrote: > Slava, > > These are exactly my thoughts, so I fully support you here. > I already wrote about it: > > http://apache-ignite-developers.2346864.n4.nabble.com/IP-finder-in-tests-td33322.html > But I kind of abandoned this activity. Feel

[jira] [Created] (IGNITE-10553) Investigate differences in test lists prior and after migration of core module tests to Junit 4 (follow-up to IGNITE-10175)

2018-12-05 Thread Oleg Ignatenko (JIRA)
Oleg Ignatenko created IGNITE-10553: --- Summary: Investigate differences in test lists prior and after migration of core module tests to Junit 4 (follow-up to IGNITE-10175) Key: IGNITE-10553 URL: https://issues.ap

[jira] [Created] (IGNITE-10554) .NET: Jars are not copied to target dir under .NET Core

2018-12-05 Thread Pavel Tupitsyn (JIRA)
Pavel Tupitsyn created IGNITE-10554: --- Summary: .NET: Jars are not copied to target dir under .NET Core Key: IGNITE-10554 URL: https://issues.apache.org/jira/browse/IGNITE-10554 Project: Ignite

[jira] [Created] (IGNITE-10555) Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Vyacheslav Daradur (JIRA)
Vyacheslav Daradur created IGNITE-10555: --- Summary: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder' Key: IGNITE-10555 URL: https://issues.apache.org/jira

Re: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Andrey Mashenkov
Slava, +1 for your proposal. Is there any ticket for this? Denis, I've just read in nabble thread you suggest to allow multicast finder for multiJVM tests and I'd think we shouldn't use multicast in test at all (excepts multicast Ip finder self tests of course), but e.g. add an assertion to force

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
Folks, let me explain one thing which is not related much to fix itself, but it is more about how we interact. If someone will just come to the list and say it is not good commit, it is a silly solution and say to others to rework these patches - it is a road to nowhere. If someone sees the potent

Re: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Vyacheslav Daradur
I filled a task [1]. >> Slava, do you think Platforms tests can be fixed as well or one more ticket should be created? I'll try to fix them within one ticket, it should be investigated a bit deeper. I'll inform about the task's progress in this thread later. Thanks! [1] https://issues.apache.o

[jira] [Created] (IGNITE-10556) Attempt to decrypt data records during read-only metastorage recovery leads to NPE

2018-12-05 Thread Pavel Kovalenko (JIRA)
Pavel Kovalenko created IGNITE-10556: Summary: Attempt to decrypt data records during read-only metastorage recovery leads to NPE Key: IGNITE-10556 URL: https://issues.apache.org/jira/browse/IGNITE-10556

Re: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Denis Mekhanikov
Andrey, Multi-JVM tests may also use a static IP finder, but it should use some specific port range instead of being shared. Something like 127.0.0.1:48500..48509 would do. Denis ср, 5 дек. 2018 г. в 18:34, Vyacheslav Daradur : > I filled a task [1]. > > >> Slava, do you think Platforms tests c

Re: Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 'TcpDiscoveryMulticastIpFinder'

2018-12-05 Thread Dmitriy Pavlov
++1 ср, 5 дек. 2018 г. в 18:38, Denis Mekhanikov : > Andrey, > > Multi-JVM tests may also use a static IP finder, but it should use some > specific port range instead of being shared. > Something like 127.0.0.1:48500..48509 would do. > > Denis > > ср, 5 дек. 2018 г. в 18:34, Vyacheslav Daradur :

[GitHub] ignite pull request #5582: for TC tests

2018-12-05 Thread daradurvs
GitHub user daradurvs opened a pull request: https://github.com/apache/ignite/pull/5582 for TC tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/daradurvs/ignite ignite-10555 Alternatively you can review and apply these chan

[jira] [Created] (IGNITE-10557) Control.sh validate index work long and broke down

2018-12-05 Thread Alexand Polyakov (JIRA)
Alexand Polyakov created IGNITE-10557: - Summary: Control.sh validate index work long and broke down Key: IGNITE-10557 URL: https://issues.apache.org/jira/browse/IGNITE-10557 Project: Ignite

[jira] [Created] (IGNITE-10558) MVCC: IgniteWalReader test failed.

2018-12-05 Thread Andrew Mashenkov (JIRA)
Andrew Mashenkov created IGNITE-10558: - Summary: MVCC: IgniteWalReader test failed. Key: IGNITE-10558 URL: https://issues.apache.org/jira/browse/IGNITE-10558 Project: Ignite Issue Type: B

[GitHub] ignite pull request #5583: IGNITE-10462: Fix Standalone wal iterator.

2018-12-05 Thread AMashenkov
GitHub user AMashenkov opened a pull request: https://github.com/apache/ignite/pull/5583 IGNITE-10462: Fix Standalone wal iterator. You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-10558 Alternativel

[GitHub] ignite pull request #5583: IGNITE-10462: Fix Standalone wal iterator.

2018-12-05 Thread AMashenkov
Github user AMashenkov closed the pull request at: https://github.com/apache/ignite/pull/5583 ---

[GitHub] ignite pull request #5583: IGNITE-10462: Fix Standalone wal iterator.

2018-12-05 Thread AMashenkov
GitHub user AMashenkov reopened a pull request: https://github.com/apache/ignite/pull/5583 IGNITE-10462: Fix Standalone wal iterator. You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-10558 Alternativ

[GitHub] ignite pull request #5584: IGNITE-8227: Proper NoOpHandler injection.

2018-12-05 Thread nizhikov
GitHub user nizhikov opened a pull request: https://github.com/apache/ignite/pull/5584 IGNITE-8227: Proper NoOpHandler injection. You can merge this pull request into a Git repository by running: $ git pull https://github.com/nizhikov/ignite IGNITE-8227-fix Alternatively you

Re: Default failure handler was changed for tests

2018-12-05 Thread Nikolay Izhikov
Dmitriy. I think we should avoid copy paste code instead of thinking about Apache Way all the time :) Anyway, I propose to return to the code! I think we should use some kind of marker base class for a cases with NoOpHandler. This has several advantages, comparing with current implementation: 1.

[GitHub] ignite pull request #5522: IGNITE-10437 GridCacheWriteBehindStoreMultithread...

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/5522 ---

[GitHub] ignite pull request #5547: IGNITE-10442

2018-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/5547 ---

[jira] [Created] (IGNITE-10559) MVCC TX: Use extracted partitions to reduce target nodes for Query Update

2018-12-05 Thread Igor Seliverstov (JIRA)
Igor Seliverstov created IGNITE-10559: - Summary: MVCC TX: Use extracted partitions to reduce target nodes for Query Update Key: IGNITE-10559 URL: https://issues.apache.org/jira/browse/IGNITE-10559

Re: [RESULT] [VOTE] Apache Ignite 2.7.0 Release (RC2)

2018-12-05 Thread Denis Magda
When are going to complete the post-release steps and announce the release? Once the binaries are available on the website we need to release the docs. I'm ready to prepare a blog post for blogs.apache.org. -- Denis On Mon, Dec 3, 2018 at 11:14 PM Nikolay Izhikov wrote: > Sorry, Alex. > > I m

Re: Default failure handler was changed for tests

2018-12-05 Thread Dmitriy Pavlov
Thanks, as an improvement to the code, this may be better. But still, it is not a reason to revert. And Anton mentioned something with better exception handling/logging. Probably we will see an implementation as well. This case here is a big thing related to The Apache Way, - and I'll explain why

Re: [RESULT] [VOTE] Apache Ignite 2.7.0 Release (RC2)

2018-12-05 Thread Nikolay Izhikov
Yes, Denis. I will do my best to complete release steps today. The only things that blocks me is bintray permissions. They required to upload rpm and deb packages to corresponding repository. I've created ticket to the INFRA [1] Seems, they, resolve it eventually. We discussed this issue with ot

Re: Default failure handler was changed for tests

2018-12-05 Thread Nikolay Izhikov
> Thanks, as an improvement to the code, this may be better. I can prepare a full patch for NoOp handler. What do you think? Anton Vinogradov, do you agree with this approach? ср, 5 дек. 2018 г. в 20:33, Dmitriy Pavlov : > Thanks, as an improvement to the code, this may be better. But still,

Re: Default failure handler was changed for tests

2018-12-05 Thread Nikolay Izhikov
Dmitriy. > The closest analog to Noop handler is mute of test failure. > By this commit, we had unmuted (possible) failures in ~5-~100=~49900 tests, and we’re still concerned about style or minor details if no-op was copy-pasted, aren’t we? Can you explain this idea a bit more? I don't unders

[jira] [Created] (IGNITE-10560) MVCC: Data loss after rebalance

2018-12-05 Thread Roman Kondakov (JIRA)
Roman Kondakov created IGNITE-10560: --- Summary: MVCC: Data loss after rebalance Key: IGNITE-10560 URL: https://issues.apache.org/jira/browse/IGNITE-10560 Project: Ignite Issue Type: Bug

[jira] [Created] (IGNITE-10561) MVCC: Unexpected transaction state after rebalance

2018-12-05 Thread Roman Kondakov (JIRA)
Roman Kondakov created IGNITE-10561: --- Summary: MVCC: Unexpected transaction state after rebalance Key: IGNITE-10561 URL: https://issues.apache.org/jira/browse/IGNITE-10561 Project: Ignite I

[jira] [Created] (IGNITE-10562) [TC Bot] DB contains invalid values for build references ID maps to value with other ID

2018-12-05 Thread Dmitriy Pavlov (JIRA)
Dmitriy Pavlov created IGNITE-10562: --- Summary: [TC Bot] DB contains invalid values for build references ID maps to value with other ID Key: IGNITE-10562 URL: https://issues.apache.org/jira/browse/IGNITE-10562

[GitHub] ignite pull request #5585: MVCC: Create "Cache 8" test suite for MVCC mode.

2018-12-05 Thread rkondakov
GitHub user rkondakov opened a pull request: https://github.com/apache/ignite/pull/5585 MVCC: Create "Cache 8" test suite for MVCC mode. You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-10367 Alterna

[jira] [Created] (IGNITE-10563) Allow manual fsync for WAL

2018-12-05 Thread Stanislav Lukyanov (JIRA)
Stanislav Lukyanov created IGNITE-10563: --- Summary: Allow manual fsync for WAL Key: IGNITE-10563 URL: https://issues.apache.org/jira/browse/IGNITE-10563 Project: Ignite Issue Type: Impro

  1   2   >