Skip to content

NIFI-2032 port for 0.x#554

Closed
olegz wants to merge 1 commit intoapache:0.xfrom
olegz:NIFI-2032-0.x
Closed

NIFI-2032 port for 0.x#554
olegz wants to merge 1 commit intoapache:0.xfrom
olegz:NIFI-2032-0.x

Conversation

@olegz
Copy link
Copy Markdown
Contributor

@olegz olegz commented Jun 21, 2016

No description provided.

@alopresto
Copy link
Copy Markdown
Contributor

I am getting test failures:

Results :

Failed tests:
  KafkaPublisherTest.validateSuccessfulSendAsDelimited:106 expected:<3> but was:<-1>
  KafkaPublisherTest.validateSuccessfulSendAsWhole:79 expected:<0> but was:<-1>
  KafkaPublisherTest.validateWithNonDefaultPartitioner:250 null

Tests in error:
  KafkaPublisherTest.validateRetries:146 » ConsumerTimeout
  KafkaPublisherTest.validateWithMultiByteCharactersNoDelimiter:231 » ConsumerTimeout

Do these run successfully for you locally? I'm going to try re-running without parallel execution.

asfgit pushed a commit that referenced this pull request Jun 22, 2016
Squashed commits:
[a9b32c6] Fixed checkstyle issues in TestStandardControllerServiceProvider.

This closes #554.

Signed-off-by: Andy LoPresto <alopresto@apache.org>
@alopresto
Copy link
Copy Markdown
Contributor

I fixed an issue in GetSNMPTest (a local variable was referenced from within an inner class, and wasn't declared final). I forgot to squash it with the commit that closed the PR. It was commit aeb07124d13e09a18a2f3085269f614a5d1ef710.

@alopresto
Copy link
Copy Markdown
Contributor

I'm going to rebase, squash, and re-push.

asfgit pushed a commit that referenced this pull request Jun 22, 2016
… (+1 squashed commit)

Squashed commits:
[c6109d4] NIFI-2032 port for 0.x (+1 squashed commit)
Squashed commits:
[a9b32c6] Fixed checkstyle issues in TestStandardControllerServiceProvider.

This closes #554.

Signed-off-by: Andy LoPresto <alopresto@apache.org>
@olegz olegz closed this Jun 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants