[jira] [Commented] (NIFI-2275) PutAzureEventHub Processor Should expose partition key

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

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

Github user ijokarumawak commented on the issue:

    https://github.com/apache/nifi/pull/676
  
    Hello @apsaltis thanks for contributing the PR!
    Excuse us for taking long for you to get any feedback. One reason I guess is that the PR title doesn't have corresponding JIRA number, we usually look for the JIRA having 'patch available' state to review. I searched the JIRA and found [NIFI-2275](https://issues.apache.org/jira/browse/NIFI-2275).
    
    If NIFI-2275 is the right one, please update the PR title, and update the JIRA by clicking the 'Submit Patch' button to proceed the JIRA to patch available state.
    
    I reviewed the code briefly, though. One thing I would like you to update is the name of the new property descriptor, related comment and code. 'Partition Key' and 'Partition Id' are used inconsistent way. I think those all should be 'Partition Key'. It seems the actual partition Id will be determined by EventHubClient library by hashing the partition key.
    
    I wonder if it'd be more useful to support expression language with Partition Key, since the processor accepts incoming flow files, user may want to assign Partition Key based on flow file attributes.
    
    Currently, PutAzureEventHub processor doesn't have Test class. I'd be happy to have one even if it only does simple property checks. It'd be a good starting point and make subsequent contribution easier.
    
    Also, Travis build failed with an unrelated error. Please rebase the branch with the latest master so that it can pass Travis build. 
    
    Thanks!


> PutAzureEventHub Processor Should expose partition key
> ------------------------------------------------------
>
>                 Key: NIFI-2275
>                 URL: https://issues.apache.org/jira/browse/NIFI-2275
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Core Framework
>    Affects Versions: 1.0.0
>            Reporter: Andrew Psaltis
>            Assignee: Andrew Psaltis
>
> Currently the PutAzureEventHub does not allow the user to set a partition key, nor does it use a default key for messages that are being sent. Therefore, the processor should do two things:
> 1. Provide a property that allows the user to set a partition key
> 2. If the partition key property is not set to use a guid as the default. This will allow the data to be evenly spread amongst the partitions.
> 3. Emit the partition key as attribute so that it is available to other processors and in the provenance data. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)