arkadius commented on pull request #4773:
URL: https://github.com/apache/nifi/pull/4773#issuecomment-810896312


   > Could you please give us some examples of formats that can be problematic?
   I've tested this change on about ~40 flows and find out 3 formats that were 
incompatible and need change:
   - `yyyy-mm-dd HH:mm:ss` -> `yyyy-MM-dd HH:mm:ss` (don't ask me how it used 
to work with `SimpleDateFormat` :)
   - `yyyy-MM-dd'T'HH:mm:ss.SSSX` -> `yyyy-MM-dd'T'HH:mm:ss.SSSXXX`
   - `dd/MMM/yyyy:HH:mm:ss` with input: `28/Jan/2021:14:58:00 +0100` -> 
`dd/MMM/yyyy:HH:mm:ss X`
   
   > though I did not use really advanced formats
   Yes, it looks like it were mainly some advanced usages of formatting zone
   
   > it is questionable to add this change in a minor release
   I think that putting this change in minor release is not a good idea. Even 
if it breaks compatibility only in those advanced cases, I would be surprised 
if it will be introduced in minor release. Do we have some other approach for 
changes for next major realse? Should I rebase it to some other branch than 
`main`? 


-- 
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]


Reply via email to