badalprasadsingh opened a new issue, #1798:
URL: https://github.com/apache/iceberg-go/issues/1798

   ### Apache Iceberg version
   
   main (development)
   
   ### Please describe the bug 🐞
   
   When `NewMetadataBuilder(formatVersion)` builds a Metadata Builder with 
`base` left `nil` (only `MetadataBuilderFromBase` sets it). Calling 
`AddSnapshot` or `AddSnapshotUpdate` on such a builder panics:
   
   ```bash
   panic: runtime error: invalid memory address or nil pointer dereference
   table.(*MetadataBuilder).addSnapshotInternal
       table/metadata.go:522
   ```
   
   Reason for this : `addSnapshotInternal` unconditionally calls 
`b.base.LastUpdatedMillis()` to compute `masTS`, with no `nil` check - unlike 
`currentNextRowID()`, which already guards the same `b.base` field.
   
   ### Status
   
   Working on the fix


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to