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
this test is no longer valid, and recursive calls, even simple ones:
f <- function(x) {
   if(x > 0) {
     f(x - 1)
   } else {
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
Therefore, keeping this test active doesn't make sense, as it will lead
to continuous test failures, when `testthat` is updated
( / SPARK-23435).
### Does this PR introduce any user-facing change?
### How was this patch tested?
Existing tests.
CC falaki
Closes #27363 from zero323/SPARK-29777-FOLLOWUP.
Authored-by: zero323 <> Signed-off-by: Dongjoon
Hyun <>
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)