rdblue commented on a change in pull request #2293:
URL: https://github.com/apache/iceberg/pull/2293#discussion_r602507337
##########
File path: core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java
##########
@@ -109,6 +109,7 @@ public ThisT set(String property, String value) {
}
protected PartitionSpec writeSpec() {
+ Preconditions.checkState(spec != null, "No data or delete files have been
added.");
Review comment:
We normally note the operation that is failing, like "Cannot determine
partition spec: no data or delete files have been added". Also, no need for
ending punctuation.
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
