Github user apiri commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178694856
No worries! Just wanted to make sure we weren't missing anything.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178694317
@apiri Sorry! It's my bad. I looked into a wrong location.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user apiri commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178671457
@PuspenduBanerjee Not sure I understand. From the linked PR in Travis I
looked at one of the build logs
(https://s3.amazonaws.com/archive.travis-ci.org/jobs/106415261/log.t
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178667386
@apiri Still have doubt.
A push fires a PR build .If that build doesn't run Test Cases , things can
become scary after merging with main line.
---
If your p
Github user apiri commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178648447
@PuspenduBanerjee Certainly a bit confusing, but this is the standard
sequence that Travis employs when building Maven based projects. This is for
accumulating all dependen
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178634682
Please find _mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V_
near line 170 in all build at [my
PR](https://travis-ci.org/apache/nifi/builds/10641525
Github user joewitt commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178627040
definitely not intended. Can you send me a link? We've seen much better
results with it now. There have been failures but they have been correct
failures.
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-178626208
@joewitt Seeing that Travis build has started to skip tests. Is that a
intended decision?
I am asking as I am not seeing such configuration in .travis.yml.
-
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-177439597
Moved to : https://github.com/apache/nifi/pull/197
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user PuspenduBanerjee closed the pull request at:
https://github.com/apache/nifi/pull/186
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-176814383
Closing PR, as found some issue in Integration test. Shall raise a new PR
later.
---
If your project is set up for it, you can reply to this email and have your
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-176439666
Hello @joewitt
PFA sample camel contexts.
[sample-application-contexts.zip](https://github.com/apache/nifi/files/108925/sample-application-contexts.
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-176232391
@apiri Again OOM in travis:
testExecuteIngestAndUpdate(org.apache.nifi.processors.standard.TestExecuteStreamCommand)
Time elapsed: 1.717 sec <<< ERROR!
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51128900
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,240 @@
+
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51128670
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,240 @@
+
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51128130
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,240 @@
+
Github user PuspenduBanerjee commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51092009
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,2
Github user PuspenduBanerjee commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51090275
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,2
Github user PuspenduBanerjee commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51090056
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,2
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-175781324
@apiri Thanks. It has built correctly with intermittent Heap issue. :+1:
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user PuspenduBanerjee commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51009342
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,2
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51004665
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,240 @@
+
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51003529
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,240 @@
+
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51002310
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,240 @@
+
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/186#discussion_r51000926
--- Diff:
nifi-nar-bundles/nifi-camel-bundle/nifi-camel-processors/src/main/java/org/apache/nifi/processors/camel/CamelProcessor.java
---
@@ -0,0 +1,240 @@
+
Github user apiri commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-175689828
@PuspenduBanerjee This was a commit that had reliance on repositories
outside of those as part of the core build and has been reverted, more details
are available in #147.
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-175688480
Hello,
Now my build is failing in Travis CI, where as it's running fine locally.
Can anyone please help
[INFO] BUILD FAILURE
[INFO]
--
Github user joewitt commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-175069513
Ok understood. So given this initial feedback it sounds like you're still
working on the PR and now have some things to work for it. When you are ready
to submit again c
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-174913731
Hello Joe,
Thanks for the review.
About Licensing, I am trying to add some maven plugin to automate that
process and strongly believe that it should b
Github user joewitt commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-174512618
Hello
The community is working toward closing down on the 050 release so PR
review bandwidth is limited right now. This PR represents a pretty significant
step t
Github user PuspenduBanerjee commented on the pull request:
https://github.com/apache/nifi/pull/186#issuecomment-174412341
Any plan to pull this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
GitHub user PuspenduBanerjee opened a pull request:
https://github.com/apache/nifi/pull/186
Nifi-Camel Integration Initial Commit
It's initial working version of nifi-camel integration.
As per feature request NIFI-924
You can merge this pull request into a Git repository by run
32 matches
Mail list logo