dentiny opened a new pull request, #1278:
URL: https://github.com/apache/iceberg-rust/pull/1278

   ## What changes are included in this PR?
   
   This is the error message I get on my end:
   ```sh
   Error: DataInvalid => Content type of entry PositionDeletes should have 
DataContentType::Data
   ```
   At first sight, I have no idea what this means, until I read the sanity 
check code.
   
   In this PR, I updated the error message for manifest write sanity check to 
provide more context:
   - What's the manifest content file
   - What's the filepath, this is useful for devs to directly understand what's 
the data file having issues
   - What's the expected and actual data content type
   
   ## Are these changes tested?
   
   No-op change


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to