arkadius commented on PR #7169: URL: https://github.com/apache/nifi/pull/7169#issuecomment-1512122120
I rebased this branch as we discussed in the original PR, but I see that meantime NiFi code become much more dependent on `DateFormat`. I see many public methods in `DataTypeUtils` taking or returning `DateFormat`. I see some transition methods like `DateTypeUtils.getDateTimeFormatter`. It makes me feel that: - I don't see the wider picture of where you already use `DateFormat` now and how you planning to use it in the future - I don't know what is the plan to get rid of `DateFormat` because it looks like there is some - I don't know if you want to use a lenient, case insensitive format in all these places or some other one If you want to draw me a wider picture I can try to continue this change but otherwise, I don't want to introduce some ambiguous behaviour in date format handling so I'll close this PR. -- 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. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
