Github user mcgilman commented on the issue:
https://github.com/apache/nifi-fds/pull/1
Will review...
---
Github user mcgilman commented on the issue:
https://github.com/apache/nifi-registry/pull/1
Awesome thanks @bbende! Will review...
---
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/611
Thanks @alopresto! The changes look good. I have verified the changes
locally in a secured cluster.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/610
Looks good @markap14! This has been 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
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/611
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/610
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/594
Thanks @scottyaslan! This has been 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
Github user mcgilman closed the pull request at:
https://github.com/apache/nifi/pull/603
---
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 mcgilman opened a pull request:
https://github.com/apache/nifi/pull/603
NIFI-1781: Updating UI to respect access controls outside of canvas
- Updating UI according to permissions through out the application.
- Shuffling provenance events, template, and cluster search
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/593
@markap14 Looks great! I made one additional change to populate snippet
results off of the actual components instead of just the snippet contents
(since the contents are now pruned).
https
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/592#discussion_r69003620
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/controller/ControllerFacade.java
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/589
@jtstorck looks good. However, I noticed that the revision version was
being necessarily checked. Those are already verified in the Tenant and
AccessPolicy resource. Also, I updated the error message
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/590
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/588
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/577
Thanks @scottyaslan! This has been merged into 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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/582
I've merged this to master. Looks good @jtstorck!
---
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/581
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 mcgilman opened a pull request:
https://github.com/apache/nifi/pull/579
NIFI-1554: Changing process for cluster detection
- Updating cluster detection to support access through the flow resource.
- Using the correct resource when authorizing a root group port during
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/577
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 mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/572#discussion_r68293664
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/http/replication
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/566
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 mcgilman opened a pull request:
https://github.com/apache/nifi/pull/567
NIFI-1554: Continuing to incorporate component based authorization
- Addressing access controls for the Controller resource.
- Addressing access controls for RAW site to site clients
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/559
Thanks @scottyaslan!
---
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/523
@scottyaslan Looks great!
The additions made since the initial PR have really improved the
performance of various parts. I've merged this to master.
---
If your project is set up
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/517
Looks good @markap14
Had one minor issue ensuring controller services are copied as part of a
process group dto. I tweak your commit to include the additionally
functionality.
---
If your
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/511
Looks great @YolandaMDavis
I'm merged this into 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
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/541#discussion_r67739760
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/service
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/541#discussion_r67739510
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/service
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/507
Looks great @bbende 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
enabled
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/526#discussion_r67553151
--- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/Group.java
---
@@ -24,7 +24,7 @@
/**
* A group that users can belong
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/540
Looks good! Thanks for knocking this out!
---
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/540
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/511
@YolandaMDavis While reading over your PR I noticed that the
ControllerService and ReportingTask URI's in the
StandandNiFiWebConfigurationContext are incorrect. They shouldn't contain the
path
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/511#discussion_r67456561
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/511#discussion_r67456463
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/pom.xml
---
@@ -355,10 +355,5 @@
spock-core
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/532
@jvwing Just addressed the issue and rebased to current 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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/532
@jvwing Actually, I think I see what the issue is. Going to verify my
change and if all looks good, I'll update the PR.
---
If your project is set up for it, you can reply to this email and have
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/532
@jvwing Thanks for jumping in to review! I am not aware of this specific
issue though I know we have a number of outside JIRAs surrounding templates and
the 1.0.0 release. It doesn't surprise me
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/517
@markap14 When exporting a template with controller services at multiple
levels, it appears the services all end up at the top level group. These should
probably retain the same scoping as the source
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/535
NIFI-2041: Unable to refresh Controller Services when scoped by Process
Group
- Fixing issue when attempting to refresh Controller Services when scoped
by Process Group.
You can merge this pull
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/532
NIFI-1941: Child group contents in exported templates
- Ensuring child group contents are included in exported templates.
You can merge this pull request into a Git repository by running:
$ git
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/531
NIFI-2007: Restoring bulletin functionality
- Restoring bulletin functionality.
- Ensuring appropriate merging of bulletins in clustered responses.
You can merge this pull request into a Git
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/526#discussion_r66968920
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/resources/nifi-web-api-context.xml
---
@@ -114,7 +123,23
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/526
@jtstorck Lets knock out these couple trivial items. Looks good so far.
---
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
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/526#discussion_r66962729
--- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/Group.java
---
@@ -24,7 +24,7 @@
/**
* A group that users can belong
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/526#discussion_r66960292
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/resources/nifi-web-api-context.xml
---
@@ -114,7 +123,23
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/526#discussion_r66959842
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/UsersResource.java
---
@@ -0,0
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/526#discussion_r66959717
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/UserGroupsResource.java
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/526#discussion_r66959598
--- 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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/515
@olegz I believe that we're going to need to introduce the templateId or
inceptionId into the flow.xml not just the template DTO. The reasoning is that
we need to support the following case
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/518
It appears that the unit tests I have added take a long time to complete
and causes Travis to timeout and fail. Specifically, the slow running tests
stand up an embedded Jetty server and deploys NiFi
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/526
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 mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/507#discussion_r66836344
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/Resources.java
---
@@ -0,0
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/524
NIFI-1781: UI updates
- Listening for window resize events more selectively.
- Fixing malformed request when configuring remote process group ports.
- Fixing malformed request when starting
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/523
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 mcgilman opened a pull request:
https://github.com/apache/nifi/pull/518
NIFI-1901: Component based access control tests
- Building component based access control tests for Connections, Funnels,
Labels, Input Ports, Output Ports, Processors, and Process Groups
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/473
@bbende The updates for ensuring thread-safe access looks good as do the
converting of existing authorized-users.xml.
+1 Merging to master.
---
If your project is set up for it, you can
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/488
This has been merged into master. Discovered a few new items and identified
a few old items that we'll still need to knock out. Another item that we still
need to tackle is ensuring that all node
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/488
Also just created another PR to consider building something to automated
the conversion of the nifi.properties [1].
[1] https://issues.apache.org/jira/browse/NIFI-1967
---
If your project
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/488
@markap14
Ran into another issue around cluster coordinator selection. When the
selected coordinator has a unheritable flow it is treated as the 'golden' copy
as they are the coordinator
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/488
@markap14
Just ran into an issue with the fields that are used in the flow
fingerprint. I think we need to reconsider which fields are included.
Specifically I had modified the penalty
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/488
@markap14
Ran into a small issue with node (re)connection regarding Templates. It
appears that Templates local to that node are replaced with the cluster set of
Templates
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/488
@markap14 I've been reviewing this PR and have some initial findings.
As I'm reviewing I'm updating various comments and naming that still
reflect the previous clustering model. I've gone
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/488
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 mcgilman closed the pull request at:
https://github.com/apache/nifi/pull/482
---
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/482
This has been merged into 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 mcgilman closed the pull request at:
https://github.com/apache/nifi/pull/472
---
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 mcgilman commented on the issue:
https://github.com/apache/nifi/pull/472
Will do. 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
enabled and wishes so
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/485
+1
Looks good. Verified functionality exporting/importing templates across 0.x
and 1.x versions. Merged to master.
---
If your project is set up for it, you can reply to this email and have
Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/485
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 mcgilman opened a pull request:
https://github.com/apache/nifi/pull/489
NIFI-1951: Custom UIs
- Removing deprecated NiFiWebContext and related classes.
- Adding authorization to Custom UIs.
- Fixing issue when creating ControllerService inline.
- Addressing
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/482
NIFI-1928: Remove registration
- Fixing UI action for bulk delete.
- Removing registration form.
- Fixing default visibility of the anonymous user warning.
You can merge this pull request
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/473
@bbende Just realized that we'll need to introduce some thread safety here.
The RevisionManager in the web tier will handle locking during the
User/Group/Policy CRUD operations. However
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/471#issuecomment-222539446
Other changes look good! Will merge to master.
During my testing of a 1.x template in an 0.x instance I discovered a
separate issue that the templates from an 1
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/471#discussion_r65097776
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/471#discussion_r65096361
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/471#discussion_r65095238
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/424#issuecomment-222531466
@YolandaMDavis I'm having some issues saving the JSON specification in the
Custom UI while running in a clustered instance. When I click save, the value
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/472
NIFI-1265: Upgrading to Jetty 9.3
This PR should only be applied to master as Jetty 9.3 requires Java 8.
You can merge this pull request into a Git repository by running:
$ git pull https
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/471#issuecomment-12578
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
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/470
NIFI-1554: Updating Cluster endpoints
- Refactoring Cluster endpoints.
- Updating frontend to accommodate clustering endpoints.
- Remove the 'Make Primary' action.
This is phase one
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/469
NIFI-1800: Providing access to Controller Services based on a given context
- UI style updates to make the components stand out better.
- Reusing controller service table in different contexts
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/452#issuecomment-221086451
@bbende The updated PR looks good to me. A number of great iterations here.
Definitely think this is a solid foundation to continue the policy based
authorizers managed
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/461#issuecomment-221051044
Thanks @markap14
All changes have been applied.
---
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 mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64262070
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProcessGroupResource.java
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64259949
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/FlowResource.java
---
@@ -321,6
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64251051
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java
---
@@ -46,6
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64250889
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64250267
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64250769
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java
---
@@ -46,6
GitHub user mcgilman opened a pull request:
https://github.com/apache/nifi/pull/461
NIFI-1781: UI authorization updates
NIFI-1781:
- Including access policies in the breadcrumb's trail.
- Updating toolbox according to group access policies.
- Updating actions in palette
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/452#issuecomment-220469545
@alopresto Great comment about the UserService. The reason we cannot do
that directly is the extension point that is getting discovered is an
Authorizer. However
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/452#issuecomment-220457062
With the builders included, looks good to me.
@jtstorck @alopresto Thoughts?
---
If your project is set up for it, you can reply to this email and have your
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/452#discussion_r63956868
--- Diff:
nifi-api/src/main/java/org/apache/nifi/authorization/AccessPolicy.java ---
@@ -0,0 +1,291 @@
+/*
+ * Licensed to the Apache Software
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/454#issuecomment-220330762
@markap14 When canceling revision claims for Connectable components in the
verifyXxx() methods, DAO access will not return null so you do not need to
check
Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/454#discussion_r63874050
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-optimistic-locking/src/main/java/org/apache/nifi/web/revision
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/452#issuecomment-220307255
@alopresto Thanks for the very thorough review! This API is designed
strictly for handling the persistence of access policies which would also
include Users and Groups
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/452#issuecomment-220025580
+1 LGTM
---
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 mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/452#issuecomment-220023481
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
Github user mcgilman commented on the pull request:
https://github.com/apache/nifi/pull/424#issuecomment-219881237
@YolandaMDavis Yep. We're definitely on the same page here. However, we
currently have not formally established any APIs/components/widgets for UI
extensions to consume
1 - 100 of 200 matches
Mail list logo