rahil-c commented on code in PR #729:
URL: https://github.com/apache/incubator-xtable/pull/729#discussion_r2176553002


##########
xtable-core/pom.xml:
##########
@@ -110,6 +110,19 @@
             <scope>test</scope>
         </dependency>
 
+        <dependency>
+            <groupId>io.delta</groupId>
+            <artifactId>delta-kernel-api</artifactId>
+            <version>4.0.0</version>

Review Comment:
   Can you add a properly in the root pom called         
<delta.kernel.version>4.0.0</delta.kernel.version>, instead of using the 
hardcoded value?
   
   Also curious how you ended up choosing delta kernel version, is there some 
specific version that needs to align with delta lake version we have in the 
repo?



-- 
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: commits-unsubscr...@xtable.apache.org

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

Reply via email to