Hi Oleg,
Yes, makes perfect sense. Thank you.
On Mon, Dec 10, 2018 at 10:14 PM oignatenko wrote:
> Hi Vovan,
>
> I just created JIRA ticket to address your concerns:
> - https://issues.apache.org/jira/browse/IGNITE-10629
>
> In brief, the plan is that a week or two after migration is over we wi
My point is - close your outdated PRs/if you don't need. It is not a
pattern nor enforcement nor bureaucracy.
You're absolutely free to use PRs in any way you like to (e.g. experiments
with TC runs).
Now count of PRs shocks newcomers.
вт, 11 дек. 2018 г. в 10:10, Nikolay Izhikov :
> Hello, Ivan
Hello, Ivan.
Personally, I keep my PR's clear.
So, I don't have dozens of opened PR.
But, I don't support Dmitriy proposal for several reasons:
1. We introduce some new, not required, level of bureaucracy.
From my experience - not required bureaucracy is a BAD thing.
2. We spread our work patte
Hi Igniters,
Infra changed notifications, so now GitHub emails are being sent to
notifications@
Web UI: https://lists.apache.org/list.html?notificati...@ignite.apache.org
You may subscribe to the list notifications-subscr...@ignite.apache.org
Sincerely,
Dmitriy Pavlov
ср, 5 дек. 2018 г. в 12:5
Hi Ivan,
Unfortunately, I don't have the necessary rights to close PRs. Only PR
author can close.
I can only imagine the following way: Produce fake commit(s) with message
Fixes #. But this may be acceptable if PR author is not responding.
Simple expiration policy may have its own issues. Th
Andrey created IGNITE-10635:
---
Summary: Ignite throws org.apache.ignite.IgniteCheckedException
Key: IGNITE-10635
URL: https://issues.apache.org/jira/browse/IGNITE-10635
Project: Ignite
Issue Type: B
Agree with Dmitriy.
We use GitHub PRs in our workflow, therefore we should keep them in order.
We can close PRs that refer to closed tickets, this can be done with a
simple script.
On Tue, Dec 11, 2018 at 9:15 AM Павлухин Иван wrote:
> Nikolay,
>
> I must say that when I first saw 1K+ open PRs
Pavel Tupitsyn created IGNITE-10634:
---
Summary: .NET: EntityFramework Core Second Level Cache
Key: IGNITE-10634
URL: https://issues.apache.org/jira/browse/IGNITE-10634
Project: Ignite
Issue
Hi Igniters,
I've detected some new issue on TeamCity to be handled. You are more than
welcomed to help.
If your changes can lead to this failure(s): We're grateful that you were a
volunteer to make the contribution to this project, but things change and you
may no longer be able to finalize
Nikolay,
I must say that when I first saw 1K+ open PRs my first thought was
that something was wrong with a review process. In my mind in not very
big project open PR list can reflect very well the real work in
progress. For bigger projects things become more complicated.
Dmitriy,
Do you have so
Oleg Ignatenko created IGNITE-10633:
---
Summary: CacheEntryProcessorExternalizableFailedTest fails on
Teamcity when executed under JUnit 4
Key: IGNITE-10633
URL: https://issues.apache.org/jira/browse/IGNITE-10633
Vasiliy Sisko created IGNITE-10632:
--
Summary: Web console: Failed to show cache mode on query notebook
page.
Key: IGNITE-10632
URL: https://issues.apache.org/jira/browse/IGNITE-10632
Project: Ignite
Nikolay, Petr,
I have already registered the package `pyignite` on PyPI[1]. The person
who is going to take the responsibility of maintaining it should create
an account on PyPI and mail me in private, so that I can grant them the
necessary rights. They also must install twine[3].
The proces
Sergey Kozlov created IGNITE-10631:
--
Summary: examples/sql/world.sql doesn't work for IgniteJdbcDriver
Key: IGNITE-10631
URL: https://issues.apache.org/jira/browse/IGNITE-10631
Project: Ignite
Sergey Kozlov created IGNITE-10630:
--
Summary: Missed descriptions for ML packages
Key: IGNITE-10630
URL: https://issues.apache.org/jira/browse/IGNITE-10630
Project: Ignite
Issue Type: Bug
dspavlov opened a new pull request #98: IGNITE-10620: Tracked branches & Issue
detector unit tests
URL: https://github.com/apache/ignite-teamcity-bot/pull/98
This is an automated message from the Apache Git Service.
To respo
NodeJs client:
--
Instruction -
https://github.com/nobitlost/ignite/blob/ignite--docs/modules/platforms/nodejs/README.md#publish-ignite-nodejs-client-on-npmjscom-instruction
Uploaded - https://www.npmjs.com/package/apache-ignite-client
PHP client:
---
Instruction -
http
Forwarding to the dev list.
*Prachi*, could you please help us with 3-5? It requires minor updates on
the webpage.
*Sergey K.*, *Peter*, could you assist with 1 and 2?
--
Denis
On Fri, Dec 7, 2018 at 11:19 PM Nikolay Izhikov wrote:
> Hello, Igniters.
>
> Seems that our download page doesn't m
Hi Vovan,
I just created JIRA ticket to address your concerns:
- https://issues.apache.org/jira/browse/IGNITE-10629
In brief, the plan is that a week or two after migration is over we will run
code inspection that detects JUnit 3 style tests that lack @Test annotation
and fix these tests if there
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/4895
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/4055
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/4229
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3981
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/4007
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3980
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3964
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3922
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3930
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3963
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3906
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3866
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3875
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3664
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3855
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3433
---
Github user alex-plekhanov closed the pull request at:
https://github.com/apache/ignite/pull/3413
---
Oleg Ignatenko created IGNITE-10629:
---
Summary: Migration follow up: check for old style tests that could
be slipped through in transition period
Key: IGNITE-10629
URL: https://issues.apache.org/jira/browse/IGNIT
Hello,
I would like add few minor clarifications to what Ed wrote.
1. After transition to JUnit4 is fully over (ETA this week) there won't be a
need to use @RunWith(JUnit4.class) anymore, so you will be writing test
classes as a normal JUnit 4 user, without any special scaffolding.
1.1. Please ke
zzzadruga opened a new pull request #97: IGNITE-10628 Add support for nightly
RunAll
URL: https://github.com/apache/ignite-teamcity-bot/pull/97
This is an automated message from the Apache Git Service.
To respond to the mess
Nikolai Kulagin created IGNITE-10628:
Summary: Add support for nightly RunAll
Key: IGNITE-10628
URL: https://issues.apache.org/jira/browse/IGNITE-10628
Project: Ignite
Issue Type: Sub-tas
Github user andrey-kuznetsov closed the pull request at:
https://github.com/apache/ignite/pull/4547
---
Github user andrey-kuznetsov closed the pull request at:
https://github.com/apache/ignite/pull/5092
---
Github user andrey-kuznetsov closed the pull request at:
https://github.com/apache/ignite/pull/5219
---
Github user andrey-kuznetsov closed the pull request at:
https://github.com/apache/ignite/pull/5541
---
Evgenii Zhuravlev created IGNITE-10627:
--
Summary: Support custom preferences like date format and other
similar features
Key: IGNITE-10627
URL: https://issues.apache.org/jira/browse/IGNITE-10627
Got it. Thank you.
пн, 10 дек. 2018 г. в 20:52, Dmitriy Pavlov :
> Vladimir, AFAIK there will be no single change for overall migration. All
> migration will be in a step-by-step manner and mostly transparent for
> contributors.
>
> Both approaches 3&4 will be available for some time.
>
> First o
Vladimir.
Since we are talking about internal component(Spark Data Frame integration)
can we use same technique as JdbcRequestHandler to obtain columns metadata?
Seems, code above will fetch up to date columns info:
org.apache.ignite.internal.processors.odbc.jdbc.JdbcRequestHandler#getColumnsMeta
Github user nizhikov closed the pull request at:
https://github.com/apache/ignite/pull/5584
---
Evgenii Zhuravlev created IGNITE-10626:
--
Summary: Save authenticated Webconsole session for more than one
page refresh
Key: IGNITE-10626
URL: https://issues.apache.org/jira/browse/IGNITE-10626
Pr
Vladimir, AFAIK there will be no single change for overall migration. All
migration will be in a step-by-step manner and mostly transparent for
contributors.
Both approaches 3&4 will be available for some time.
First of all, we need an opportunity to run existing integration tests in
JUnit4.
пн,
Vovan,
1) Tomorrow in core-module.
2) I believe that nothing, old JUnit3 would work if they are a new one.
Otherwise, they need to fix merge conflict and add @Test on their new test.
And I think that it's unavoidable, we couldn't ask all people to finish
their current activity and not start a new
Ed,
Several questions from my side:
1) When the change is expected to be merged?
2) What contributors with opened PRs and new or updated JUnit3 tests are
supposed to do? Rewrite to JUnit4?
If yes, then we should give them time to have a chance to get used to new
approach and resolve possible conf
Ivan,
So, suggested actions with the new approach:
1. Add @Test annotation on test methods.
2. Add @RunWith(JUnit4.class) annotation on test class.
3. Add @Before, @After on methods which should run before, after a
test (setUp, tearDown in current approach).
4. Add your test-class to a suite using
Pavel Kovalenko created IGNITE-10625:
Summary: Do first checkpoint on node start before join to topology
Key: IGNITE-10625
URL: https://issues.apache.org/jira/browse/IGNITE-10625
Project: Ignite
Github user dspavlov closed the pull request at:
https://github.com/apache/ignite/pull/1892
---
Github user dspavlov closed the pull request at:
https://github.com/apache/ignite/pull/2276
---
GitHub user Jokser opened a pull request:
https://github.com/apache/ignite/pull/5629
IGNITE-10624 Update deployment id for recovered cache after join to topology
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ign
The main concern is related to chances that newcomer will have to obtain a
review support from the community.
Actually, a lot of people doing their best to provide a feedback to
newcomers, and count of issues still in PA state goes down (84 is a
relatively small count of issues in PA state). But 1
Pavel Kovalenko created IGNITE-10624:
Summary: Cache deployment id may be different that cluster-wide
after recovery
Key: IGNITE-10624
URL: https://issues.apache.org/jira/browse/IGNITE-10624
Proje
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/2121
---
Hello, Dmitriy.
What, exactly concerns newcomers?
What is wrong with opened PR?
How project will benefit from closed PR?
> The same proposal is related to IEP statuses. If you were involved in an
IEP, please validate its status
+1. We should maintain IEP description up to date.
пн, 10 дек. 2018
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/3190
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/2986
---
asfgit closed pull request #95: IGNITE-9542: new run history stripe
URL: https://github.com/apache/ignite-teamcity-bot/pull/95
This is an automated message from the Apache Git Service.
To respond to the message, please log on
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/2482
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/1955
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/1922
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/1544
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/1493
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/3144
---
dspavlov commented on a change in pull request #95: IGNITE-9542: new run
history stripe
URL: https://github.com/apache/ignite-teamcity-bot/pull/95#discussion_r240287246
##
File path:
ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/issue/IssueDetector.java
##
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/2779
---
zzzadruga commented on a change in pull request #95: IGNITE-9542: new run
history stripe
URL: https://github.com/apache/ignite-teamcity-bot/pull/95#discussion_r240279974
##
File path:
ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/issue/IssueDetector.java
##
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/1522
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/1481
---
Github user EdShangGG closed the pull request at:
https://github.com/apache/ignite/pull/4239
---
Github user zstan closed the pull request at:
https://github.com/apache/ignite/pull/3919
---
Hi Igniters,
Newcomers to Apache Ignite sometimes became concerned about many open PRs
in the project. Apache Ignite TC Bot also performs runs checks with a PR
open. Apache Ignite pulls list https://github.com/apache/ignite/pulls contains
1442 PRs open while only 84 issues are waiting for review.
GitHub user akalash opened a pull request:
https://github.com/apache/ignite/pull/5628
IGNITE-10622 to sent pending messages when new next node was selected
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ig
GitHub user rkondakov opened a pull request:
https://github.com/apache/ignite/pull/5627
IGNITE-10464: MVCC: Create "PDS 4" 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-10
GitHub user AMashenkov opened a pull request:
https://github.com/apache/ignite/pull/5626
IGNITE-10359: Add deployment feature for EntryProcessor.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-10359
GitHub user voropava reopened a pull request:
https://github.com/apache/ignite/pull/5425
IGNITE-10300 Fixed correct message in case of user/password retries eâ¦
â¦xceeded.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain
GitHub user antonovsergey93 opened a pull request:
https://github.com/apache/ignite/pull/5625
IGNITE-8717 Cache configuration was moved to metastore
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-87
GitHub user Jokser opened a pull request:
https://github.com/apache/ignite/pull/5624
IGNITE-10556 Skip records deserialization on recovery
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-10556
Alter
GitHub user EdShangGG opened a pull request:
https://github.com/apache/ignite/pull/5623
IGNITE-10623 Possible JVM crash when node stopped with errors
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-1
GitHub user AMashenkov opened a pull request:
https://github.com/apache/ignite/pull/5622
IGNITE-10472: MVCC: Add Dependency Injection support for EntryProcessor.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ign
Eduard Shangareev created IGNITE-10623:
--
Summary: Possible JVM crash in case of stopping node with errors
Key: IGNITE-10623
URL: https://issues.apache.org/jira/browse/IGNITE-10623
Project: Ignite
Reverted.
https://issues.apache.org/jira/browse/IGNITE-8227 reopened
пн, 10 дек. 2018 г. в 16:23, Dmitriy Pavlov :
> Anton, I was expecting that you revert, because you wanted to do it.
>
> Provided that I agree that fix could be reverted because of both
> functional and style possible improveme
GitHub user voropava opened a pull request:
https://github.com/apache/ignite/pull/5621
Ignite 2.5.1 p162
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-2.5.1-p162
Alternatively you can review and a
Github user voropava closed the pull request at:
https://github.com/apache/ignite/pull/5425
---
Github user voropava closed the pull request at:
https://github.com/apache/ignite/pull/5563
---
Anton Kalashnikov created IGNITE-10622:
--
Summary: Undelivered ensure message to some nodes
Key: IGNITE-10622
URL: https://issues.apache.org/jira/browse/IGNITE-10622
Project: Ignite
Issue
Anton, I was expecting that you revert, because you wanted to do it.
Provided that I agree that fix could be reverted because of both functional
and style possible improvements, does not mean I believe it is the only
option and it should be reverted.
Even if I agree to revert doesn't mean all com
GitHub user ygerzhedovich opened a pull request:
https://github.com/apache/ignite/pull/5620
IGNITE-10621
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-10621
Alternatively you can review and apply
Github user asfgit closed the pull request at:
https://github.com/apache/ignite/pull/5590
---
Github user devozerov closed the pull request at:
https://github.com/apache/ignite/pull/5593
---
Github user devozerov closed the pull request at:
https://github.com/apache/ignite/pull/5572
---
Github user asfgit closed the pull request at:
https://github.com/apache/ignite/pull/4685
---
Yury Gerzhedovich created IGNITE-10621:
--
Summary: Track all running queries
Key: IGNITE-10621
URL: https://issues.apache.org/jira/browse/IGNITE-10621
Project: Ignite
Issue Type: Task
asfgit closed pull request #94: Visa History page fix
URL: https://github.com/apache/ignite-teamcity-bot/pull/94
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 provenance:
As this is a foreign pull request (f
1 - 100 of 125 matches
Mail list logo
