exceptionfactory commented on PR #9379: URL: https://github.com/apache/nifi/pull/9379#issuecomment-2414288784
Thanks for the thorough review @dan-s1. I pushed an update to address the comments. I updated comments in several places, adding handling for `null` Cells, and added a new unit test method to exercise the Timestamp inference behavior for `STRING` Cell Types. I marked several threads as resolved, but if there are any where you think additional feedback is needed, feel free to unresolve and continue the conversation. Otherwise, hopefully we can wrap this up soon. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
