arkadius commented on pull request #4773: URL: https://github.com/apache/nifi/pull/4773#issuecomment-811074151
Minor notice, that I should mention is that I personally don't need to have this feature in some release in the near feature. In my deployment we have our own fork with a few modifications. I just want to share this with community, because in our deployment it gave us significant performance improvement. I don't think that introducing new function(s) would be a good way to go. It would complicate API for end users. Also I have other concerns: 1. What about other places like `format()` function? What about implicit string/number to date conversion? 2. What would be plan to fully migrate to `DateTimeFormatter`? Maybe better option would be putting a feature flag in config? But personally IMO good migration guide for some major release would be enough. -- 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]
