NirYanay2005 commented on PR #10877: URL: https://github.com/apache/nifi/pull/10877#issuecomment-3912777627
About the `StorageClass`, i need both and cant use one without the other so i don't mind waiting a bit more. I want to work through an improved approach, but I need a bit more direction on where the change should be made. I understand that introducing a separate IcebergRecordConverter is not ideal, and that the better location would be during Iceberg Record construction inside PutIcebergRecord. However, I’m not yet fully familiar with how PutIcebergRecord constructs the Iceberg Record object and where NiFi Record field conversion is applied. Could you point me to the specific method in PutIcebergRecord where timestamp type handling would be most appropriate? I’m happy to iterate on the implementation once I understand the intended extension point. -- 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]
