GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/722
NIFI-2399 Correcting comparison of maxEventId against lastEventId in â¦
â¦SiteToSiteProvenanceReportingTask
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/702
NIFI-1950 Updating FileAuthorizer to convert access controls from inpâ¦
â¦ut and output ports during legacy conversion
You can merge this pull request into a Git repository by running:
$ git
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/574
@YolandaMDavis rebased against master to resolve conflicts and pushed a new
commit that addresses your feedback
---
If your project is set up for it, you can reply to this email and have your
reply
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/574
@YolandaMDavis thanks for reviewing! Working on some changes based on your
feedback.
Regarding the need for RangerBasePluginWithPolicies... unfortunately the
PolicyEngine is a private member
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/574#discussion_r72070305
--- Diff:
nifi-nar-bundles/nifi-ranger-bundle/nifi-ranger-plugin/src/main/java/org/apache/nifi/ranger/authorization/RangerNiFiAuthorizer.java
---
@@ -0,0
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/724
@mcgilman thanks for reviewing, great points... i just pushed a new commit
that addresses the uniqueness across users and groups, and on updates
---
If your project is set up for it, you can reply
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/735
NIFI-2390 Separating of users and groups from authorizations.xml intoâ¦
⦠separate file
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bbende
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/719
NIFI-2389 Refactoring identity mapping and applying it to FileAuthoriâ¦
â¦zer for initial admin, cluster nodes, and legacy authorized users
You can merge this pull request into a Git repository
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/823
Tested this out with MySQL, created a table like the following:
```
mysql> CREATE TABLE BIGINT_TEST (id bigint(20) unsigned, name varchar(255));
Query OK, 0 rows affected (0.02
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/772
NIFI-2453 Making FileAuthorizer perform initial seeding when users anâ¦
â¦d groups are already present
You can merge this pull request into a Git repository by running:
$ git pull https
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/768
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/768
I'm a +1 and will merge to master. My only comment is that we may want to
call out in the release notes that we removed the auto-migration from the old
state mechanism... probably an unlikely scenario
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/826
+1 looks good, verified functionality, will merge to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/824
This looks good to me. I am able to build cleanly on a JDK without the
unlimited crypto, and I can run the toolkit to generate certs, listing the
contents of the client p12 and the keystore/truststore
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/832
This PR appears to have fixed an issue I was seeing while testing PR 829.
Full build passed with contrib-check, ran the IT test which passed, and setup a
three node cluster with no issues, so I'm a +1
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/829
I've been testing this with PR 832, which has a few minor clustering fixes,
and together everything has been working well. I have repeatedly taken an old
flow.xml that previously produced
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/843
NIFI-2553 Fixing handling of Paths in HDFS processors
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bbende/nifi NIFI-2553
Alternatively you can
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/822
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/800
Looks good, will merge to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/829
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/823
Latest update looks good, nice work! Will merge to master
---
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
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/755#discussion_r73373783
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/AccessPolicyResource.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/755#discussion_r73377622
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/AccessResource.java
---
@@ -111,7
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/755#discussion_r73405602
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProvenanceResource.java
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/755
I'm a +1 once the comments above are addressed, let me know if you want me
to merge it in after you make any updates
---
If your project is set up for it, you can reply to this email and have your
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/706
Tested out the trigger serially and appears to be working as expected. I am
a +1 to merge this in, do you want to squash the commits first?
---
If your project is set up for it, you can reply
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/780
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/784
+1 looks good, merging to master...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/784
Reviewing...
---
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
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/843#discussion_r74689944
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/AbstractHadoopProcessor.java
---
@@ -90,20
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/843#discussion_r74689871
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/AbstractHadoopProcessor.java
---
@@ -286,8
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/843
@YolandaMDavis Thanks for reviewing and testing! After looking at the
existing EL validator in StandardValidators, it wasn't used by any processor
and I believe the logic didn't fully make sense
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/665
Reviewing...
---
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
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/668
NIFI-929 Changing RunNiFi to write just the pid to nifi.pid and the fâ¦
â¦ull status to nifi.status
You can merge this pull request into a Git repository by running:
$ git pull https
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/672
@JPercivall good catch, just pushed an update to correct the variable names
---
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
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/677
NIFI-929 Changing permissions of nifi.pid to be readable by all, writâ¦
â¦able only by user
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/659
NIFI-2194 Caching metrics in AmbariReportingTask so each iteration seâ¦
â¦nds last iteration's metrics
You can merge this pull request into a Git repository by running:
$ git pull https
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/773
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/789
+1 looks good, will merge to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/773
+1 looks good, merged to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/791
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/788
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/788
Looks good, will merge to master shortly...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/800
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/706
Latest update is looking good... one thing I noticed, if you send in an
Avro file that does not have the partition columns of the table, it throws an
IOException around line 435 when trying to extract
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/755
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/706
Error handling on the relationships is looking good, thanks for making
those updates.
One more thing I ran into, when I set concurrent tasks to 2 on the
PutHiveStreamingProcessor and I
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/767
NIFI-1869 Cloning controller services when referenced by reporting taâ¦
â¦sks and upgrading from 0.x to 1.0.0
You can merge this pull request into a Git repository by running:
$ git pull https
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/706#discussion_r72677845
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/PutHiveStreaming.java
---
@@ -0,0 +1,657
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/744
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/772
@YolandaMDavis Thanks for testing this! The behavior you are seeing is
interesting, I tested this a few times before submitting the PR and again right
now and can't seem to reproduce that. Also had
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/861
I was able to use the toolkit to generate all the configs for a two-node
secure cluster and got into the UI using the generated p12. Everything looks
good, will merge to master.
---
If your project
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/875#discussion_r75010710
--- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc ---
@@ -169,6 +169,81 @@ Now that the User Interface has been secured, we can
easily secure Site
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/861
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/874
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/874
Looks good, will merge to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/862
Thanks for the contribution! Will take a look shortly and get this merged
in.
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/859
Looks good, will merge to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/860
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/859
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/869
Thanks for update, merged to master!
Just an FYI, I had a bit of trouble getting the changes to apply from the
PR onto master... for future it would be easiest to have a specific branch
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/867
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/860
+1 full build passed with contrib-check and tested a lot of combinations of
components referencing controller services to ensure i could only use
controller services i had READ access to, will merge
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/878
This looks good, will merge to master, thanks!
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1479
Reviewing...
---
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
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1527#discussion_r102721436
--- Diff:
nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-0-10-processors/src/main/java/org/apache/nifi/processors/kafka/pubsub/ConsumerPool.java
---
@@ -126,6
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1349
@joewitt sure, just squashed and pushed, thanks!
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1349
@joewitt I just ran a full build with contrib-check and passed. If you were
just building the nifi-hbase-bundle, it is possible you need to rebuild
standard services as well because the HBase
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1420
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1424
+1 Will merge
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1410
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1410
+1 Verified build passes with contrib-check, tested the Wait/Notify
processors to test original cache functionality, code looks good, will merge to
master, thanks Koji!
---
If your project is set up
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1420
@ijokarumawak this is really awesome stuff! Tested this out and working
nicely, code looks great. I made a couple of small tweaks to the documentation
on the processors, going to merge to master
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/867
Looks good, will merge to master, thanks!
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/863
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/869
@patricker thanks for the contribution! Definitely makes sense to support
both formats...
What do you think about using a regex to determine if it is a long, and if
so then parseLong
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/863
I made one small change to also evaluate the record name and num records to
analyze when doing JSON, but other than that looks good and merged to master,
thanks!
---
If your project is set up
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/834
I ran into some test failures when building this PR, possibly due to the
fact that I don't have JCE installed?
```
Failed tests:
ââ
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/920
Looks good, verified selecting from a table with a binary column no longer
produces an error, will merge to master
---
If your project is set up for it, you can reply to this email and have your
reply
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/914
Tested and all looks good, will merge to master, thanks Andrew!
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/920
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/914
Thanks for making the updates, looks good, testing it now... I noticed this
PR is against 0.x, since most new development is going into 1.x, should we put
this into master?
I think the changes
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/930
I've been reviewing this... so far I have tested ConsumeKafka and
PublishKafka (0.9.0) against an Apache Kafka 0.9.0 broker, with PLAINTEXT and
also with SSL, and both appear to be working well. Trying
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/930
Quick update... I was finally able to get a Kerberos environment setup and
did a successful test with SASL, so that covers the 0.9.0 client against the
0.9.0 broker, with PLAINTEXT, SSL, and SASL
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/930
Using the latest updates I was able to verify the 0.10 processors against
the 0.10 broker using PLAINTEXT, SSL, and SASL, and also tested the 0.9.0
processors against an 0.10 broker.
I'm a +1
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/958
+1 full build passes and provenance appears to work fine, will merge to
master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/987
I performed the same test as I did when reworking the processors...
everything worked with 0.9 plaintext, ssl, and SASL, and 0.10 plaintext, ssl,
and SASL I'll go ahead and merge.
---
If your project
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/995
+1 full build passing and verified a three node cluster starts successfully
when all flows are empty with different root group ids, will merge to master
---
If your project is set up for it, you can
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/990
@trixpan I just merged in a patch for
https://issues.apache.org/jira/browse/NIFI-2741 that includes this same
change, if you are good with it I think we can close this PR and JIRA. Sorry
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1014
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1012
+1 verified the validation can now pass, will merge to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1013
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1013
+1 Will merge to master
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1012
Reviewing...
---
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
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1014
+1 verified i can no longer change a connection when i don't have WRITE to
the parent process group, will merge to master
---
If your project is set up for it, you can reply to this email and have
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1011
+1 tested this out by simulating a small delay during refresh and tested
adding/removing out-of-order scenarios and all appear to work now, will merge
to master
---
If your project is set up
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1019
+1 verified site-to-site functionality for secure and un-secure, with raw
and http, will merge to master
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1005
NIFI-1768 Adding TLS/SSL support to Solr processors
NIFI-1980 Added a default value for PutSolrContentStream commitWithIn
NIFI-2568 Added Kerberos support to Solr processors
Upgrading
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1006
NIFI-2760 Specifying jersey-bundle 1.19 for Ranger plugin
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bbende/nifi ranger-jersey-bundle
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1056
@brosander this is really cool! Been playing with it and seems to be
working well.
I noticed in the scripts that we set the max heap to 24mb, I was wondering
if there is any concern about
1 - 100 of 897 matches
Mail list logo