You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2021/12/14 01:33:24 UTC

[GitHub] [iceberg] kbendick opened a new pull request #3734: [SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

kbendick opened a new pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734


   Backporting https://github.com/apache/iceberg/pull/3543 to Spark 3.1.


-- 
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] rdblue commented on pull request #3734: [SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

Posted by GitBox <gi...@apache.org>.
rdblue commented on pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734#issuecomment-994988779


   Thanks, @kbendick!


-- 
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] kbendick commented on pull request #3734: [DO NOT MERGE][SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

Posted by GitBox <gi...@apache.org>.
kbendick commented on pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734#issuecomment-994142422


   > Is there anything different from 3.2 we should look out?
   
   Nope. This is a direct port line for line from 3.2.
   
   The only thing temporarily was including the backport from that other PR which was merged. But now that I've rebased, it's the exact same.


-- 
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] jackye1995 commented on pull request #3734: [DO NOT MERGE][SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

Posted by GitBox <gi...@apache.org>.
jackye1995 commented on pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734#issuecomment-994138937


   Is there anything different from 3.2 we should look out?


-- 
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] kbendick commented on pull request #3734: [SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

Posted by GitBox <gi...@apache.org>.
kbendick commented on pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734#issuecomment-993076497


   It looks like `SparkTestBaseWithCatalog` wasn't back-ported. I'll do that now.


-- 
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] kbendick edited a comment on pull request #3734: [SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

Posted by GitBox <gi...@apache.org>.
kbendick edited a comment on pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734#issuecomment-993089898


   The changes for simplifying unit tests against Spark Catalogs for using only one catalog were not backported to versions of Spark < 3.2.
   
   I have backported it for Spark 3.1 in this PR: https://github.com/apache/iceberg/pull/3737
   
   I am temproarily rebasing this PR off of that branch, to ensure that the backported changes in #3737 work with this PR. I will then rebase back against master.


-- 
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] kbendick commented on pull request #3734: [SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

Posted by GitBox <gi...@apache.org>.
kbendick commented on pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734#issuecomment-993089898


   The changes for simplifying unit tests against Spark Catalogs for using only one catalog were not backported to versions of Spark < 3.2.
   
   I have backported it for Spark 3.1 in this PR: https://github.com/apache/iceberg/pull/3737


-- 
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] rdblue merged pull request #3734: [SPARK] Backport cache.expiration-interval-ms config to Spark 3.1

Posted by GitBox <gi...@apache.org>.
rdblue merged pull request #3734:
URL: https://github.com/apache/iceberg/pull/3734


   


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