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 02:12:01 UTC

[GitHub] [iceberg] kbendick opened a new pull request #3737: [SPARK] Backport SpakTestBaseWithCatalog to Spark 3.1 to simplify run…

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


   …ning tests with only one catalog
   
   This backports #3549 to Spark 3.1.
   
   This is needed so that #3734 can be reviewed, as it uses `SparkTestBaseWithCatalog`.
   
   cc @rdblue @jackye1995 


-- 
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 #3737: [SPARK] Backport SpakTestBaseWithCatalog to Spark 3.1 to simplify run…

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


   Thank you both!


-- 
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 #3737: [SPARK] Backport SpakTestBaseWithCatalog to Spark 3.1 to simplify run…

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


   cc @RussellSpitzer or @aokolnychyi if you get a chance and can merge this relatively simple backport fo me =)


-- 
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 merged pull request #3737: [SPARK] Backport SpakTestBaseWithCatalog to Spark 3.1 to simplify run…

Posted by GitBox <gi...@apache.org>.
jackye1995 merged pull request #3737:
URL: 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] kbendick commented on pull request #3737: [SPARK] Backport SpakTestBaseWithCatalog to Spark 3.1 to simplify run…

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


   The only change I didn't backport was the one test that was updated to use `SparkTestBaseWithCatalog`, which is `TestRuntimeFiltering`.
   
   That's a Spark 3.2 only test as far as I can tell, and I will be using these in https://github.com/apache/iceberg/pull/3734.
   
   I will merge this PR and that PR locally to ensure everything is ok with the two PRs together.


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