nastra commented on a change in pull request #2664:
URL: https://github.com/apache/iceberg/pull/2664#discussion_r649282810
##########
File path:
nessie/src/main/java/org/apache/iceberg/nessie/NessieTableOperations.java
##########
@@ -139,26 +131,11 @@ public FileIO io() {
return fileIO;
}
- /**
- * try and get a Spark application id if one exists.
- *
- * <p>
- * We haven't figured out a general way to pass commit messages through to
the Nessie committer yet.
- * This is hacky but gets the job done until we can have a more complete
commit/audit log.
- * </p>
- */
- private String applicationId() {
- String appId = null;
- TableMetadata current = current();
- if (current != null) {
- Snapshot snapshot = current.currentSnapshot();
- if (snapshot != null) {
- Map<String, String> summary = snapshot.summary();
- appId = summary.get("spark.app.id");
- }
-
- }
- return appId;
+ Map<String, String> getCatalogOptions() {
Review comment:
given that we're passing `catalogOptions` via the constructor, we can
remove that method again
--
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]
