Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KAFKA-2605: Replace 'catch: Throwable' clauses with 'NonFatal' #531

Closed
wants to merge 1 commit into from

Conversation

@jinxing64
Copy link
Contributor

commented Nov 15, 2015

'catch: Throwable' will catch VirtualMachineError, ThreadDeath, InterruptedException, LinkageError, ControlThrowable, NotImplementedError; we don't want to catch those kind of error

@jinxing64 jinxing64 closed this Nov 15, 2015

gwenshap added a commit to confluentinc/kafka that referenced this pull request Jul 21, 2019
CONFLUENT: Jenkins gradle exit 1 on executor failure, 0 on test failu…
…re (apache#531)

We wish to return a yellow build on a test failure, but wish to fail the
build completely when the gradle executors do not finish cleanly. This
change will achieve that.
(cherry picked from commit 334acf48d40e6a3c627cb71990fa91e1616e4921)

-- This also changes max number of forks to 4, to improve build stability
gwenshap added a commit to confluentinc/kafka that referenced this pull request Jul 21, 2019
CONFLUENT: Jenkins gradle exit 1 on executor failure, 0 on test failu…
…re (apache#531)

We wish to return a yellow build on a test failure, but wish to fail the
build completely when the gradle executors do not finish cleanly. This
change will achieve that.
(cherry picked from commit 334acf48d40e6a3c627cb71990fa91e1616e4921)
gwenshap added a commit to confluentinc/kafka that referenced this pull request Jul 21, 2019
CONFLUENT: Jenkins gradle exit 1 on executor failure, 0 on test failu…
…re (apache#531) (#215)

We wish to return a yellow build on a test failure, but wish to fail the
build completely when the gradle executors do not finish cleanly. This
change will achieve that.
(cherry picked from commit 334acf48d40e6a3c627cb71990fa91e1616e4921)

-- This also changes max number of forks to 4, to improve build stability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.