jonkeane commented on PR #12867: URL: https://github.com/apache/arrow/pull/12867#issuecomment-1098461877
Hmm, it looks like `tryInvokeRestart` isn't available prior to 3.6: https://github.com/ursacomputing/crossbow/runs/6013703013?check_suite_focus=true#step:5:21063 https://github.com/wch/r-source/blob/7d25524f8b83d63979cc18f0914a321bfbe9a819/src/library/base/R/conditions.R#L86-L92 might be helpful if we wanted to approximate the old behavior. Or maybe we should just do something like this: ``` .suppressWarnings <- suppressWarnings suppressWarnings <- function(exprs, ...) { if ({r version > 4.0.0}) { .suppressWarnings(exprs, ...) } else { .suppressWarnings(exprs) } } ``` And then we don't have to worry about the internals too much? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
