Skip to content

NIFI-3025: Bump nifi-spark-receiver's jackson version to match Spark 2.0.1#1207

Closed
randerzander wants to merge 1 commit intoapache:masterfrom
randerzander:master
Closed

NIFI-3025: Bump nifi-spark-receiver's jackson version to match Spark 2.0.1#1207
randerzander wants to merge 1 commit intoapache:masterfrom
randerzander:master

Conversation

@randerzander
Copy link
Contributor

@randerzander randerzander commented Nov 11, 2016

For all changes:

  • Is there a JIRA ticket associated with this PR? Is it referenced
    in the commit message?

  • Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

  • Has your PR been rebased against the latest commit within the target branch (typically master)?

  • Is your initial contribution a single, squashed commit?

For code changes:

  • Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?

No code or doc changes, but I ran "mvn -Pcontrib-check clean install" successfully anyway.

@randerzander
Copy link
Contributor Author

Tests all passed locally for me, also on travis-ci.. maybe the AppVeyor CI infrastructure has problems tonight?

I see two unrelated errors:

  1. NPM errors during nifi-web build -- unrelated and not sure they're even errors? <3 npm
  2. PersistentProvenanceRepository tests -- unsure about this one, but passed locally for me

@randerzander
Copy link
Contributor Author

After opening and closing the PR to re-run tests, AppVeyor failed with a different error (testSendLargeFileHTTPWithProxy).

I think it's safe to say it's a problem with AppVeyor's boxes.

@olegz
Copy link
Contributor

olegz commented Nov 14, 2016

+1 Merging

@asfgit asfgit closed this in 8d3177c Nov 14, 2016
dstreev pushed a commit to dstreev/nifi-1 that referenced this pull request Dec 9, 2016
dstreev pushed a commit to dstreev/nifi-1 that referenced this pull request Dec 9, 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