arkadius commented on pull request #4773: URL: https://github.com/apache/nifi/pull/4773#issuecomment-812406744
I see that we can't move forward with decision how to roll out this changes. You don't mind if I move discussion about both options to our dev mailing list? Summarizing I see that we have two options: 1. To modify implementation of current `format` / `toDate` functions - won't mess up api, but will need some date format modifications on users side (which will be described in tests and migration guide). Additional question is in which version should be introduced? 2. To add new `formatDateTime`, `toDateTime` functions (using `DateTimeFormatter`) next to existing `format`, `toDate` (using `SimpleDateFormat`, will be deprecated) - won't break flows using current functions, but will add some complexity in api which will vanish after removal of deprecated functions. Am I right? @turcsanyip can you take off 'requested changes' from this PR? Or there are some other changes in code left that I should made? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
