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

Daniel Stieglitz edited comment on NIFI-12100 at 9/28/23 12:42 AM:
-------------------------------------------------------------------

[~exceptionfactory] I think I figured it out. I had to name the branch 
something other than NIFI-12100 as that was used for the branch I created to 
merge into main. Also, I could not build with Java 21 but I was able to build 
with Java 17. The PR is [7803|https://github.com/apache/nifi/pull/7803]

For future knowledge what branch do I actually push to? I had done origin which 
I realize is not correct and then on github I had to switch to which branch I 
wanted to merge to.


was (Author: JIRAUSER294662):
[~exceptionfactory] I think I figured it out. I had to name the branch 
something other than NIFI-12100 as that was used for the branch I created to 
merge into main. Also, I could not build with Java 21 but I was able to build 
with Java 17. The PR is [7803|https://github.com/apache/nifi/pull/7803]

> Remove  ConvertExcelToCSVProcessor 
> -----------------------------------
>
>                 Key: NIFI-12100
>                 URL: https://issues.apache.org/jira/browse/NIFI-12100
>             Project: Apache NiFi
>          Issue Type: Improvement
>            Reporter: Daniel Stieglitz
>            Assignee: Daniel Stieglitz
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> ConvertExcelToCSVProcessor is no longer needed since there is now the 
> ExcelReader which along with CSVRecordSetWriter can be used in ConvertRecord 
> to achieve the same thing.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to