NIFI-1135:

- Adding additional parameters to be able to limit the size of the provenance response. Specifically, whether the events should be summarized and whether events should be returned incrementally before the query has completed.
- Ensuring the cluster node address is included in provenance events returned.
- Ensuring there is a cluster coordinator before attempting to get the cluster node address.
- Removing exponential back off between provenance requests.
- Ensuring the content viewer url is retrieve before initializing the provenance table.

This closes #1413.

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-client-dto/src/main/java/org/apache/nifi/web/api/dto/provenance/ProvenanceRequestDTO.java

@@ -38,6 +38,9 @@ public class ProvenanceRequestDTO {
     private String maximumFileSize;
     private Integer maxResults;
 
+    private Boolean summarize;
+    private Boolean incrementalResults;
+
     /**
      * @return the search terms to use for this search
      */
@@ -139,4 +142,34 @@ public class ProvenanceRequestDTO {
     public void setClusterNodeId(String clusterNodeId) {
         this.clusterNodeId = clusterNodeId;
     }
+
+    /**
+     * @return whether or not incremental results are returned. If false, provenance events
+     * are only returned once the query completes. This property is true by default.
+     */
+    @ApiModelProperty(
+            value = "Whether or not incremental results are returned. If false, provenance events"
+                    + " are only returned once the query completes. This property is true by default."
+    )
+    public Boolean getIncrementalResults() {
+        return incrementalResults;
+    }
+
+    public void setIncrementalResults(Boolean incrementalResults) {
+        this.incrementalResults = incrementalResults;
+    }
+
+    /**
+     * @return whether or not to summarize provenance events returned. This property is false by default.
+     */
+    @ApiModelProperty(
+            value = "Whether or not to summarize provenance events returned. This property is false by default."
+    )
+    public Boolean getSummarize() {
+        return summarize;
+    }
+
+    public void setSummarize(Boolean summarize) {
+        this.summarize = summarize;
+    }
 }

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/NiFiServiceFacade.java

@@ -206,9 +206,11 @@ public interface NiFiServiceFacade {
      * Retrieves provenance.
      *
      * @param queryId identifier
+     * @param summarize whether to summarize the event dtos
+     * @param incrementalResults whether to return any events if the search has not finished
      * @return result
      */
-    ProvenanceDTO getProvenance(String queryId);
+    ProvenanceDTO getProvenance(String queryId, Boolean summarize, Boolean incrementalResults);
 
     /**
      * Deletes provenance.

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java

@@ -2148,8 +2148,8 @@ public class StandardNiFiServiceFacade implements NiFiServiceFacade {
     }
 
     @Override
-    public ProvenanceDTO getProvenance(final String queryId) {
+    public ProvenanceDTO getProvenance(final String queryId, final Boolean summarize, final Boolean incrementalResults) {
-        return controllerFacade.getProvenanceQuery(queryId);
+        return controllerFacade.getProvenanceQuery(queryId, summarize, incrementalResults);
     }
 
     @Override

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProvenanceEventResource.java

@@ -22,7 +22,9 @@ import com.wordnik.swagger.annotations.ApiParam;
 import com.wordnik.swagger.annotations.ApiResponse;
 import com.wordnik.swagger.annotations.ApiResponses;
 import com.wordnik.swagger.annotations.Authorization;
+import org.apache.nifi.cluster.coordination.ClusterCoordinator;
 import org.apache.nifi.cluster.coordination.http.replication.RequestReplicator;
+import org.apache.nifi.cluster.protocol.NodeIdentifier;
 import org.apache.nifi.controller.repository.claim.ContentDirection;
 import org.apache.nifi.stream.io.StreamUtils;
 import org.apache.nifi.web.DownloadableContent;
@@ -286,6 +288,13 @@ public class ProvenanceEventResource extends ApplicationResource {
         final ProvenanceEventDTO event = serviceFacade.getProvenanceEvent(id.getLong());
         event.setClusterNodeId(clusterNodeId);
 
+        // populate the cluster node address
+        final ClusterCoordinator coordinator = getClusterCoordinator();
+        if (coordinator != null) {
+            final NodeIdentifier nodeId = coordinator.getNodeIdentifier(clusterNodeId);
+            event.setClusterNodeAddress(nodeId.getApiAddress() + ":" + nodeId.getApiPort());
+        }
+
         // create a response entity
         final ProvenanceEventEntity entity = new ProvenanceEventEntity();
         entity.setProvenanceEvent(event);

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProvenanceResource.java

@@ -46,6 +46,7 @@ import org.apache.nifi.web.api.entity.ProvenanceOptionsEntity;
 import javax.servlet.http.HttpServletRequest;
 import javax.ws.rs.Consumes;
 import javax.ws.rs.DELETE;
+import javax.ws.rs.DefaultValue;
 import javax.ws.rs.GET;
 import javax.ws.rs.HttpMethod;
 import javax.ws.rs.POST;
@@ -294,6 +295,17 @@ public class ProvenanceResource extends ApplicationResource {
                     required = false
             )
             @QueryParam("clusterNodeId") final String clusterNodeId,
+            @ApiParam(
+                    value = "Whether or not incremental results are returned. If false, provenance events"
+                            + " are only returned once the query completes. This property is true by default.",
+                    required = false
+            )
+            @QueryParam("summarize") @DefaultValue(value = "false") final Boolean summarize,
+            @ApiParam(
+                    value = "Whether or not to summarize provenance events returned. This property is false by default.",
+                    required = false
+            )
+            @QueryParam("incrementalResults") @DefaultValue(value = "true") final Boolean incrementalResults,
             @ApiParam(
                     value = "The id of the provenance query.",
                     required = true
@@ -314,7 +326,7 @@ public class ProvenanceResource extends ApplicationResource {
         }
 
         // get the provenance
-        final ProvenanceDTO dto = serviceFacade.getProvenance(id);
+        final ProvenanceDTO dto = serviceFacade.getProvenance(id, summarize, incrementalResults);
         dto.getRequest().setClusterNodeId(clusterNodeId);
         populateRemainingProvenanceContent(dto);
 

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/controller/ControllerFacade.java

@@ -972,7 +972,7 @@ public class ControllerFacade implements Authorizable {
         final QuerySubmission querySubmission = provenanceRepository.submitQuery(query, NiFiUserUtils.getNiFiUser());
 
         // return the query with the results populated at this point
-        return getProvenanceQuery(querySubmission.getQueryIdentifier());
+        return getProvenanceQuery(querySubmission.getQueryIdentifier(), requestDto.getSummarize(), requestDto.getIncrementalResults());
     }
 
     /**
@@ -981,7 +981,7 @@ public class ControllerFacade implements Authorizable {
      * @param provenanceId id
      * @return the results of a provenance query
      */
-    public ProvenanceDTO getProvenanceQuery(String provenanceId) {
+    public ProvenanceDTO getProvenanceQuery(String provenanceId, Boolean summarize, Boolean incrementalResults) {
         try {
             // get the query to the provenance repository
             final ProvenanceRepository provenanceRepository = flowController.getProvenanceRepository();
@@ -1027,11 +1027,14 @@ public class ControllerFacade implements Authorizable {
             provenanceDto.setPercentCompleted(queryResult.getPercentComplete());
 
             // convert each event
+            final boolean includeResults = incrementalResults == null || Boolean.TRUE.equals(incrementalResults);
+            if (includeResults || queryResult.isFinished()) {
                 final List<ProvenanceEventDTO> events = new ArrayList<>();
                 for (final ProvenanceEventRecord record : queryResult.getMatchingEvents()) {
-                events.add(createProvenanceEventDto(record));
+                    events.add(createProvenanceEventDto(record, Boolean.TRUE.equals(summarize)));
                 }
                 resultsDto.setProvenanceEvents(events);
+            }
 
             if (requestDto.getMaxResults() != null && queryResult.getTotalHitCount() >= requestDto.getMaxResults()) {
                 resultsDto.setTotalCount(requestDto.getMaxResults().longValue());
@@ -1227,7 +1230,7 @@ public class ControllerFacade implements Authorizable {
             final ProvenanceEventRecord event = flowController.replayFlowFile(originalEvent, user);
 
             // convert the event record
-            return createProvenanceEventDto(event);
+            return createProvenanceEventDto(event, false);
         } catch (final IOException ioe) {
             throw new NiFiCoreException("An error occurred while getting the specified event.", ioe);
         }
@@ -1313,7 +1316,7 @@ public class ControllerFacade implements Authorizable {
             dataAuthorizable.authorize(authorizer, RequestAction.READ, NiFiUserUtils.getNiFiUser(), attributes);
 
             // convert the event
-            return createProvenanceEventDto(event);
+            return createProvenanceEventDto(event, false);
         } catch (final IOException ioe) {
             throw new NiFiCoreException("An error occurred while getting the specified event.", ioe);
         }
@@ -1325,7 +1328,23 @@ public class ControllerFacade implements Authorizable {
      * @param event event
      * @return event
      */
-    private ProvenanceEventDTO createProvenanceEventDto(final ProvenanceEventRecord event) {
+    private ProvenanceEventDTO createProvenanceEventDto(final ProvenanceEventRecord event, final boolean summarize) {
+        final ProvenanceEventDTO dto = new ProvenanceEventDTO();
+        dto.setId(String.valueOf(event.getEventId()));
+        dto.setEventId(event.getEventId());
+        dto.setEventTime(new Date(event.getEventTime()));
+        dto.setEventType(event.getEventType().name());
+        dto.setFlowFileUuid(event.getFlowFileUuid());
+        dto.setFileSize(FormatUtils.formatDataSize(event.getFileSize()));
+        dto.setFileSizeBytes(event.getFileSize());
+        dto.setComponentId(event.getComponentId());
+        dto.setComponentType(event.getComponentType());
+
+        // sets the component details if it can find the component still in the flow
+        setComponentDetails(dto);
+
+        // only include all details if not summarizing
+        if (!summarize) {
             // convert the attributes
             final Comparator<AttributeDTO> attributeComparator = new Comparator<AttributeDTO>() {
                 @Override
@@ -1362,21 +1381,11 @@ public class ControllerFacade implements Authorizable {
                 attributes.add(attribute);
             }
 
-        // build the event dto
+            // additional event details
-        final ProvenanceEventDTO dto = new ProvenanceEventDTO();
-        dto.setId(String.valueOf(event.getEventId()));
             dto.setAlternateIdentifierUri(event.getAlternateIdentifierUri());
             dto.setAttributes(attributes);
             dto.setTransitUri(event.getTransitUri());
-        dto.setEventId(event.getEventId());
-        dto.setEventTime(new Date(event.getEventTime()));
-        dto.setEventType(event.getEventType().name());
-        dto.setFileSize(FormatUtils.formatDataSize(event.getFileSize()));
-        dto.setFileSizeBytes(event.getFileSize());
-        dto.setComponentId(event.getComponentId());
-        dto.setComponentType(event.getComponentType());
             dto.setSourceSystemFlowFileId(event.getSourceSystemFlowFileIdentifier());
-        dto.setFlowFileUuid(event.getFlowFileUuid());
             dto.setRelationship(event.getRelationship());
             dto.setDetails(event.getDetails());
 
@@ -1412,9 +1421,6 @@ public class ControllerFacade implements Authorizable {
                     && !Result.Approved.equals(replayAuthorized.getResult()) ? replayAuthorized.getExplanation() : contentAvailability.getReasonNotReplayable());
             dto.setSourceConnectionIdentifier(event.getSourceQueueIdentifier());
 
-        // sets the component details if it can find the component still in the flow
-        setComponentDetails(dto);
-
             // event duration
             if (event.getEventDuration() >= 0) {
                 dto.setEventDuration(event.getEventDuration());
@@ -1435,6 +1441,7 @@ public class ControllerFacade implements Authorizable {
             final List<String> childUuids = new ArrayList<>(event.getChildUuids());
             Collections.sort(childUuids, Collator.getInstance(Locale.US));
             dto.setChildUuids(childUuids);
+        }
 
         return dto;
     }

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/provenance/nf-provenance-lineage.js

@@ -26,8 +26,7 @@ nf.ng.ProvenanceLineage = function () {
     var config = {
         sliderTickCount: 75,
         urls: {
-            lineage: '../nifi-api/provenance/lineage',
+            lineage: '../nifi-api/provenance/lineage'
-            events: '../nifi-api/provenance-events/'
         }
     };
 
@@ -90,41 +89,6 @@ nf.ng.ProvenanceLineage = function () {
         });
     };
 
-    /**
-     * Shows the details for the specified event.
-     *
-     * @param {string} eventId
-     * @param {string} clusterNodeId    The id of the node in the cluster where this event/flowfile originated
-     */
-    var showEventDetails = function (eventId, clusterNodeId, provenanceTableCtrl) {
-        getEventDetails(eventId, clusterNodeId).done(function (response) {
-            provenanceTableCtrl.showEventDetails(response.provenanceEvent);
-        });
-    };
-
-    /**
-     * Gets the details for the specified event.
-     *
-     * @param {string} eventId
-     * @param {string} clusterNodeId    The id of the node in the cluster where this event/flowfile originated
-     */
-    var getEventDetails = function (eventId, clusterNodeId) {
-        var url;
-        if (nf.Common.isDefinedAndNotNull(clusterNodeId)) {
-            url = config.urls.events + encodeURIComponent(eventId) + '?' + $.param({
-                    clusterNodeId: clusterNodeId
-                });
-        } else {
-            url = config.urls.events + encodeURIComponent(eventId);
-        }
-
-        return $.ajax({
-            type: 'GET',
-            url: url,
-            dataType: 'json'
-        }).fail(nf.Common.handleAjaxError);
-    };
-
     /**
      * Submits the specified lineage request.
      *
@@ -792,7 +756,7 @@ nf.ng.ProvenanceLineage = function () {
                         'class': 'lineage-view-event',
                         'text': 'View details',
                         'click': function () {
-                            showEventDetails(d.id, clusterNodeId, provenanceTableCtrl);
+                            provenanceTableCtrl.showEventDetails(d.id, clusterNodeId);
                         }
                     }];
 
@@ -852,17 +816,17 @@ nf.ng.ProvenanceLineage = function () {
                             };
 
                             // polls for the event lineage
-                            var pollLineage = function (nextDelay) {
+                            var pollLineage = function () {
                                 getLineage(lineage).done(function (response) {
                                     lineage = response.lineage;
 
-                                    // process the lineage, if its not done computing wait delay seconds before checking again
+                                    // process the lineage
-                                    processLineage(nextDelay);
+                                    processLineage();
                                 }).fail(closeDialog);
                             };
 
                             // processes the event lineage
-                            var processLineage = function (delay) {
+                            var processLineage = function () {
                                 // if the request was cancelled just ignore the current response
                                 if (cancelled === true) {
                                     closeDialog();
@@ -907,13 +871,9 @@ nf.ng.ProvenanceLineage = function () {
                                         // clear the timer since we've been invoked
                                         lineageTimer = null;
 
-                                        // calculate the next delay (back off)
-                                        var backoff = delay * 2;
-                                        var nextDelay = backoff > provenanceTableCtrl.MAX_DELAY ? provenanceTableCtrl.MAX_DELAY : backoff;
-
                                         // for the lineage
-                                        pollLineage(nextDelay);
+                                        pollLineage();
-                                    }, delay * 1000);
+                                    }, 2000);
                                 }
                             };
 
@@ -934,7 +894,7 @@ nf.ng.ProvenanceLineage = function () {
                         // collapses the lineage for the specified event in the specified direction
                         var collapseLineage = function (eventId, provenanceTableCtrl) {
                             // get the event in question and collapse in the appropriate direction
-                            getEventDetails(eventId, clusterNodeId).done(function (response) {
+                            provenanceTableCtrl.getEventDetails(eventId, clusterNodeId).done(function (response) {
                                 var provenanceEvent = response.provenanceEvent;
                                 var eventUuid = provenanceEvent.flowFileUuid;
                                 var eventUuids = d3.set(provenanceEvent.childUuids);
@@ -1371,18 +1331,17 @@ nf.ng.ProvenanceLineage = function () {
                 $('#lineage-query-dialog').modal('hide');
             };
 
-            // polls the server for the status of the lineage, if the lineage is not
+            // polls the server for the status of the lineage
-            // done wait nextDelay seconds before trying again
+            var pollLineage = function (provenanceTableCtrl) {
-            var pollLineage = function (nextDelay, provenanceTableCtrl) {
                 getLineage(lineage).done(function (response) {
                     lineage = response.lineage;
 
-                    // process the lineage, if its not done computing wait delay seconds before checking again
+                    // process the lineage
-                    processLineage(nextDelay, provenanceTableCtrl);
+                    processLineage(provenanceTableCtrl);
                 }).fail(closeDialog);
             };
 
-            var processLineage = function (delay, provenanceTableCtrl) {
+            var processLineage = function (provenanceTableCtrl) {
                 // if the request was cancelled just ignore the current response
                 if (cancelled === true) {
                     closeDialog();
@@ -1417,13 +1376,9 @@ nf.ng.ProvenanceLineage = function () {
                         // clear the timer since we've been invoked
                         lineageTimer = null;
 
-                        // calculate the next delay (back off)
-                        var backoff = delay * 2;
-                        var nextDelay = backoff > provenanceTableCtrl.MAX_DELAY ? provenanceTableCtrl.MAX_DELAY : backoff;
-
                         // poll lineage
-                        pollLineage(nextDelay, provenanceTableCtrl);
+                        pollLineage(provenanceTableCtrl);
-                    }, delay * 1000);
+                    }, 2000);
                 }
             };
 
@@ -1432,7 +1387,7 @@ nf.ng.ProvenanceLineage = function () {
                 lineage = response.lineage;
 
                 // process the results, if they are not done wait 1 second before trying again
-                processLineage(1, provenanceTableCtrl);
+                processLineage(provenanceTableCtrl);
             }).fail(closeDialog);
         }
     }

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/provenance/nf-provenance-table.js

@@ -34,7 +34,7 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
             searchOptions: '../nifi-api/provenance/search-options',
             replays: '../nifi-api/provenance-events/replays',
             provenance: '../nifi-api/provenance',
-            provenanceEvents: '../nifi-api/provenance-events',
+            provenanceEvents: '../nifi-api/provenance-events/',
             clusterSearch: '../nifi-api/flow/cluster/search-results',
             d3Script: 'js/d3/d3.min.js',
             lineageScript: 'js/nf/provenance/nf-provenance-lineage.js',
@@ -57,7 +57,7 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
         var eventId = $('#provenance-event-id').text();
 
         // build the url
-        var dataUri = config.urls.provenanceEvents + '/' + encodeURIComponent(eventId) + '/content/' + encodeURIComponent(direction);
+        var dataUri = config.urls.provenanceEvents + encodeURIComponent(eventId) + '/content/' + encodeURIComponent(direction);
 
         // perform the request once we've received a token
         nf.Common.getAccessToken(config.urls.downloadToken).done(function (downloadToken) {
@@ -736,7 +736,7 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
                 }
             } else if (provenanceGrid.getColumns()[args.cell].id === 'moreDetails') {
                 if (target.hasClass('show-event-details')) {
-                    provenanceTableCtrl.showEventDetails(item);
+                    provenanceTableCtrl.showEventDetails(item.eventId, item.clusterNodeId);
                 }
             }
         });
@@ -864,7 +864,9 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
         var provenanceEntity = {
             'provenance': {
                 'request': $.extend({
-                    maxResults: config.maxResults
+                    maxResults: config.maxResults,
+                    summarize: true,
+                    incrementalResults: false
                 }, provenance)
             }
         };
@@ -889,7 +891,14 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
         var url = provenance.uri;
         if (nf.Common.isDefinedAndNotNull(provenance.request.clusterNodeId)) {
             url += '?' + $.param({
-                    clusterNodeId: provenance.request.clusterNodeId
+                    clusterNodeId: provenance.request.clusterNodeId,
+                    summarize: true,
+                    incrementalResults: false
+                });
+        } else {
+            url += '?' + $.param({
+                    summarize: true,
+                    incrementalResults: false
                 });
         }
 
@@ -1003,11 +1012,6 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
 
     function ProvenanceTableCtrl() {
 
-        /**
-         * The max delay between requests.
-         */
-        this.MAX_DELAY = 4;
-
         /**
          * The server time offset
          */
@@ -1147,21 +1151,19 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
                 $('#provenance-query-dialog').modal('hide');
             };
 
-            // polls the server for the status of the provenance, if the provenance is not
+            // polls the server for the status of the provenance
-            // done wait nextDelay seconds before trying again
+            var pollProvenance = function () {
-            var pollProvenance = function (nextDelay) {
                 getProvenance(provenance).done(function (response) {
                     // update the provenance
                     provenance = response.provenance;
 
                     // process the provenance
-                    processProvenanceResponse(nextDelay);
+                    processProvenanceResponse();
                 }).fail(closeDialog);
             };
 
-            // processes the provenance, if the provenance is not done wait delay
+            // processes the provenance
-            // before polling again
+            var processProvenanceResponse = function () {
-            var processProvenanceResponse = function (delay) {
                 // if the request was cancelled just ignore the current response
                 if (cancelled === true) {
                     closeDialog();
@@ -1193,13 +1195,9 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
                         // clear the timer since we've been invoked
                         provenanceTimer = null;
 
-                        // calculate the next delay (back off)
-                        var backoff = delay * 2;
-                        var nextDelay = backoff > self.MAX_DELAY ? self.MAX_DELAY : backoff;
-
                         // poll provenance
-                        pollProvenance(nextDelay);
+                        pollProvenance();
-                    }, delay * 1000);
+                    }, 2000);
                 }
             };
 
@@ -1209,16 +1207,43 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
                 provenance = response.provenance;
 
                 // process the results, if they are not done wait 1 second before trying again
-                processProvenanceResponse(1);
+                processProvenanceResponse();
             }).fail(closeDialog);
         },
 
+        /**
+         * Gets the details for the specified event.
+         *
+         * @param {string} eventId
+         * @param {string} clusterNodeId    The id of the node in the cluster where this event/flowfile originated
+         */
+        getEventDetails: function (eventId, clusterNodeId) {
+            var url;
+            if (nf.Common.isDefinedAndNotNull(clusterNodeId)) {
+                url = config.urls.provenanceEvents + encodeURIComponent(eventId) + '?' + $.param({
+                        clusterNodeId: clusterNodeId
+                    });
+            } else {
+                url = config.urls.provenanceEvents + encodeURIComponent(eventId);
+            }
+
+            return $.ajax({
+                type: 'GET',
+                url: url,
+                dataType: 'json'
+            }).fail(nf.Common.handleAjaxError);
+        },
+
         /**
          * Shows the details for the specified action.
          *
-         * @param {object} event
+         * @param {string} eventId
+         * @param {string} clusterNodeId    The id of the node in the cluster where this event/flowfile originated
          */
-        showEventDetails: function (event) {
+        showEventDetails: function (eventId, clusterNodeId) {
+            provenanceTableCtrl.getEventDetails(eventId, clusterNodeId).done(function (response) {
+                var event = response.provenanceEvent;
+
                 // update the event details
                 $('#provenance-event-id').text(event.eventId);
                 $('#provenance-event-time').html(nf.Common.formatValue(event.eventTime)).ellipsis();
@@ -1426,6 +1451,7 @@ nf.ng.ProvenanceTable = function (provenanceLineageCtrl) {
 
                 // show the dialog
                 $('#event-details-dialog').modal('show');
+            });
         }
     }
 

nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/provenance/nf-provenance.js

@@ -83,7 +83,7 @@ nf.ng.Provenance = function (provenanceTableCtrl) {
      */
     var loadAbout = function () {
         // get the about details
-        $.ajax({
+        return $.ajax({
             type: 'GET',
             url: config.urls.about,
             dataType: 'json'