You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2021/05/14 17:14:54 UTC

[GitHub] [iceberg] shenodaguirguis commented on pull request #2496: [#2039] Support default value semantics - API changes

shenodaguirguis commented on pull request #2496:
URL: https://github.com/apache/iceberg/pull/2496#issuecomment-841386263


   hi @RussellSpitzer , @rymurr , I worked on a prototype to verify the planned changes will fix the issue, and I created a draft-PR to share, for reference. This prototype  gives an idea how things will look like.  I also tested complex types in this prototype Here it is: https://github.com/linkedin/iceberg/pull/70
   (note: I skipped the ser/deser code changes in that prototype).  
   
   I have just pushed a new commit to this PR, after addressing the comments. This also reflects the commit we merged to the linkedin branch.  Please take a look.
   CC: @wmoustafa 


-- 
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org