[ 
https://issues.apache.org/jira/browse/NIFI-3709?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16292648#comment-16292648
 ] 

ASF GitHub Bot commented on NIFI-3709:
--------------------------------------

Github user ijokarumawak commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2335#discussion_r157219892
  
    --- Diff: 
nifi-nar-bundles/nifi-atlas-bundle/nifi-atlas-reporting-task/src/main/java/org/apache/nifi/atlas/provenance/StandardAnalysisContext.java
 ---
    @@ -0,0 +1,136 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one or more
    + * contributor license agreements.  See the NOTICE file distributed with
    + * this work for additional information regarding copyright ownership.
    + * The ASF licenses this file to You under the Apache License, Version 2.0
    + * (the "License"); you may not use this file except in compliance with
    + * the License.  You may obtain a copy of the License at
    + *
    + *     http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +package org.apache.nifi.atlas.provenance;
    +
    +import org.apache.nifi.atlas.NiFiFlow;
    +import org.apache.nifi.atlas.resolver.ClusterResolver;
    +import org.apache.nifi.authorization.user.NiFiUser;
    +import org.apache.nifi.controller.status.ConnectionStatus;
    +import org.apache.nifi.provenance.ProvenanceEventRecord;
    +import org.apache.nifi.provenance.ProvenanceRepository;
    +import org.apache.nifi.provenance.lineage.ComputeLineageResult;
    +import org.apache.nifi.provenance.lineage.ComputeLineageSubmission;
    +import org.slf4j.Logger;
    +import org.slf4j.LoggerFactory;
    +
    +import java.io.IOException;
    +import java.util.Collections;
    +import java.util.List;
    +import java.util.Set;
    +import java.util.concurrent.TimeUnit;
    +
    +public class StandardAnalysisContext implements AnalysisContext {
    +
    +    private final Logger logger = 
LoggerFactory.getLogger(StandardAnalysisContext.class);
    +    private final NiFiFlow nifiFlow;
    +    private final ClusterResolver clusterResolver;
    +    private final ProvenanceRepository provenanceRepository;
    +
    +    public StandardAnalysisContext(NiFiFlow nifiFlow, ClusterResolver 
clusterResolver,
    +                                   ProvenanceRepository 
provenanceRepository) {
    +        this.nifiFlow = nifiFlow;
    +        this.clusterResolver = clusterResolver;
    +        this.provenanceRepository = provenanceRepository;
    +    }
    +
    +    @Override
    +    public List<ConnectionStatus> findConnectionTo(String componentId) {
    +        return nifiFlow.getIncomingRelationShips(componentId);
    +    }
    +
    +    @Override
    +    public List<ConnectionStatus> findConnectionFrom(String componentId) {
    +        return nifiFlow.getOutgoingRelationShips(componentId);
    +    }
    +
    +    @Override
    +    public String getNiFiClusterName() {
    +        return nifiFlow.getClusterName();
    +    }
    +
    +    @Override
    +    public ClusterResolver getClusterResolver() {
    +        return clusterResolver;
    +    }
    +
    +    private ComputeLineageResult getLineageResult(long eventId, 
ComputeLineageSubmission submission) {
    +        final ComputeLineageResult result = submission.getResult();
    +        try {
    +            if (result.awaitCompletion(10, TimeUnit.SECONDS)) {
    --- End diff --
    
    I agree on that this can be a costly operation. The reason to query 
provenance is to compute a lineage from a DROP provenance event. This is used 
by 'Complete Path' strategy. I wrote documentation on performance impact. If 
this does not work for a use-case, then user can choose another strategy, 
'Simple Path'. Simple Path does not query provenance events this way. It 
analyzes each individual event, so should be more lightweight.


> Export NiFi flow dataset lineage to Apache Atlas
> ------------------------------------------------
>
>                 Key: NIFI-3709
>                 URL: https://issues.apache.org/jira/browse/NIFI-3709
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Extensions
>            Reporter: Koji Kawamura
>            Assignee: Koji Kawamura
>
> While Apache NiFi has provenance and event level lineage support within its 
> data flow, Apache Atlas also does manage lineage between dataset and process 
> those interacting with such data. 
> It would be beneficial for users who use both NiFi and Atlas and if they can 
> see end-to-end data lineage on Atlas lineage graph, as some type of dataset 
> are processed by both NiFi and technologies around Atlas such as Storm, 
> Falcon or Sqoop. For example, Kafka topics and Hive tables.
> In order to make this integration happen, I propose a NiFi reporting task 
> that analyzes NiFi flow then creates DataSet and Process entities in Atlas.
> The challenge is how to design NiFi flow dataset level lineage within Atlas 
> lineage graph.
> If we just add a single NiFi process and connect every DataSet from/to it, it 
> would be too ambiguous since it won't be clear which part of a NiFi flow 
> actually interact with certain dataset.
> But if we put every NiFi processor as independent process in Atlas, it would 
> be too granular, too. Also, we already have detailed event level lineage in 
> NiFi, we wouldn't need the same level in Atlas.
> If we can group certain processors in a NiFI flow as a process in Atlas, it 
> would be a nice granularity.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to