exceptionfactory commented on PR #10877:
URL: https://github.com/apache/nifi/pull/10877#issuecomment-3926945793

   > 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.
   
   Thanks for the reply. I recommend tracing through `PutIcebergRecord` and 
looking at `DelegatedRecord` as a starting 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]

Reply via email to