ForeverAngry commented on code in PR #428:
URL: https://github.com/apache/incubator-xtable/pull/428#discussion_r1588603231
##########
core/src/main/java/org/apache/xtable/delta/DeltaConversionTarget.java:
##########
@@ -268,9 +281,23 @@ private void commitTransaction() {
}
private Map<String, String> getConfigurationsForDeltaSync() {
+
+ // The output of detail() method has only one row with the following
schema.
+ // deltaTable.detail() is added to the constructor for this class, and
sets
+ // a private variable to of deltaTable
+
+ // limit the results to the attributes needed
+ Dataset<Row> record = deltaTable.detail().select("minWriterVersion",
"minReaderVersion");
Review Comment:
I dont think so. I looked through the api for the
`deltaLog.snapshot().metadata()`, and did not find a referance to it. The only
place i found it at was here:
[](https://docs.delta.io/latest/delta-utility.html)
--
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]
