You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "snazy (via GitHub)" <gi...@apache.org> on 2023/01/23 17:50:08 UTC

[GitHub] [iceberg] snazy opened a new pull request, #6649: Nessie-build: add test dependencies

snazy opened a new pull request, #6649:
URL: https://github.com/apache/iceberg/pull/6649

   Future change in Nessie requires these test dependencies, which are no longer transitive.
   
   The change hasn't been merged to Nessie yet, but once it's merged, it would break our Nessie/Iceberg integration testing. It's a rather no-op for Iceberg.


-- 
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: issues-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] snazy closed pull request #6649: Nessie-build: add test dependencies

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy closed pull request #6649: Nessie-build: add test dependencies
URL: https://github.com/apache/iceberg/pull/6649


-- 
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: issues-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] nastra commented on a diff in pull request #6649: Nessie-build: add test dependencies

Posted by "nastra (via GitHub)" <gi...@apache.org>.
nastra commented on code in PR #6649:
URL: https://github.com/apache/iceberg/pull/6649#discussion_r1085444775


##########
build.gradle:
##########
@@ -668,6 +668,8 @@ project(':iceberg-nessie') {
     implementation "com.fasterxml.jackson.core:jackson-core"
 
     testImplementation "org.projectnessie:nessie-jaxrs-testextension"
+    testImplementation "org.projectnessie:nessie-versioned-persist-in-memory"

Review Comment:
   it feels to me that this should be updated once the nessie version is bumped?



-- 
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: issues-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org