james-willis commented on code in PR #2077:
URL: https://github.com/apache/sedona/pull/2077#discussion_r2201910882


##########
.github/workflows/java.yml:
##########
@@ -65,6 +65,9 @@ jobs:
           - spark: 4.0.0
             scala: 2.13.8
             jdk: '17'
+          - spark: 3.5.4

Review Comment:
   since i fixed the jpostal java 17 thing it isnt strictly. I left it because 
It matches AWS Glue 5.0 and I think thats a good piece of test coverage.



##########
pom.xml:
##########
@@ -422,6 +422,18 @@
             <id>Spark Packages</id>
             <url>https://repos.spark-packages.org/</url>
         </repository>
+<!--        TODO remove me and use jpostal actual release-->
+        <repository>

Review Comment:
   yesh only just now did I finish publishing the release version



-- 
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]

Reply via email to