FailedChanges

Summary

  1. [SPARK-29777][FOLLOW-UP][SPARKR] Remove no longer valid test for (details)
  2. Revert "[SPARK-29777][FOLLOW-UP][SPARKR] Remove no longer valid test for (details)
Commit 81ea5a4cc1617de0dbbc61811847320724b3644f by gurwls223
[SPARK-29777][FOLLOW-UP][SPARKR] Remove no longer valid test for
recursive calls
### What changes were proposed in this pull request?
Disabling test for cleaning closure of recursive function.
### Why are the changes needed?
As of
https://github.com/apache/spark/commit/9514b822a70d77a6298ece48e6c053200360302c
this test is no longer valid, and recursive calls, even simple ones:
```lead
f <- function(x) {
   if(x > 0) {
     f(x - 1)
   } else {
     x
   }
}
```
lead to
``` Error: node stack overflow
```
This is issue is silenced when tested with `testthat` 1.x (reason
unknown), but cause failures when using `testthat` 2.x (issue can be
reproduced outside test context).
Problem is known and tracked by
[SPARK-30629](https://issues.apache.org/jira/browse/SPARK-30629)
Therefore, keeping this test active doesn't make sense, as it will lead
to continuous test failures, when `testthat` is updated
(https://github.com/apache/spark/pull/27359 / SPARK-23435).
### Does this PR introduce any user-facing change?
No.
### How was this patch tested?
Existing tests.
CC falaki
Closes #27363 from zero323/SPARK-29777-FOLLOWUP.
Authored-by: zero323 <mszymkiewicz@gmail.com> Signed-off-by: Dongjoon
Hyun <dhyun@apple.com>
The file was modifiedR/pkg/tests/fulltests/test_utils.R (diff)
Commit 5f1cb2fe86b1b624c8f89121f12e01c5539bdafe by gurwls223
Revert "[SPARK-29777][FOLLOW-UP][SPARKR] Remove no longer valid test for
recursive calls"
This reverts commit 81ea5a4cc1617de0dbbc61811847320724b3644f.
The file was modifiedR/pkg/tests/fulltests/test_utils.R (diff)