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

ASF GitHub Bot commented on MINIFICPP-682:
------------------------------------------

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

    https://github.com/apache/nifi-minifi-cpp/pull/448#discussion_r239104013
  
    --- Diff: nanofi/include/api/nanofi.h ---
    @@ -68,60 +94,173 @@ typedef int c2_start_callback(char *);
     
     void enable_async_c2(nifi_instance *, C2_Server *, c2_stop_callback *, 
c2_start_callback *, c2_update_callback *);
     
    +/**
    + * Creates a new, empty flow
    + * @param instance the instance new flow will belong to
    + * @return a pointer to the created flow
    + **/
    +flow *create_new_flow(nifi_instance * instance);
     
    -uint8_t run_processor(const processor *processor);
    -
    -flow *create_new_flow(nifi_instance *);
     
    -flow *create_flow(nifi_instance *, const char *);
    +/**
    + * Creates new flow and adds the first processor in case a valid name is 
provided
    + * @deprecated  as there is no proper indication of processor adding 
errors,
    + * usage of "create_new_flow" and "add_processor is recommended instead
    + * @param instance the instance new flow will belong to
    + * @param first_processor name of the first processor to be instanciated
    + * @attention in case first processor is empty or doesn't name any 
existing processor, an empty flow is returned.
    + * @return a pointer to the created flow
    + **/
    +DEPRECATED flow *create_flow(nifi_instance * instance, const char * 
first_processor);
     
    -flow *create_getfile(nifi_instance *instance, flow *parent, GetFileConfig 
*c);
    +/**
    + * Add a getfile processor to "parent" flow.
    + * Creates new flow in instance in case "parent" is nullptr
    + * @deprecated as getfile processor can be added using "add_processor" 
function,
    --- End diff --
    
    This is precisely something we want to keep by design. Let's talk about 
this before merging this. 


> C API: provide functions to create custom processors
> ----------------------------------------------------
>
>                 Key: MINIFICPP-682
>                 URL: https://issues.apache.org/jira/browse/MINIFICPP-682
>             Project: NiFi MiNiFi C++
>          Issue Type: Improvement
>            Reporter: Arpad Boda
>            Assignee: Arpad Boda
>            Priority: Major
>              Labels: CAPI, nanofi
>             Fix For: 0.6.0
>
>
> Extend C API to:
> -Provide functions that can be used used to implement custom processor.
> -Custom processor should be able to read/update both the content and the 
> attributes of flowfile, route to "failure" and "success" relationships. 
> -API should support adding these custom processors to flows and invoke them 
> as standalones, too. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to