Commit a1a6981f authored by Orvid King's avatar Orvid King Committed by Facebook Github Bot

Don't attempt to change the memory limit in the large Retrying test on Windows

Summary: The `(get|set)rlimit` stub implementations don't support setting memory limits, and I have no intention of implementing it, so disable it, as there are no meaningful memory limits on Windows to begin with.

Reviewed By: yfeldblum

Differential Revision: D9569424

fbshipit-source-id: 738d73f90eb33957ca4a80cb9b3c568034ef141e
parent 062f62a6
......@@ -190,6 +190,7 @@ TEST(RetryingTest, policy_sleep_defaults) {
}
TEST(RetryingTest, large_retries) {
#ifndef _WIN32
rlimit oldMemLimit;
PCHECK(getrlimit(RLIMIT_AS, &oldMemLimit) == 0);
......@@ -203,6 +204,7 @@ TEST(RetryingTest, large_retries) {
SCOPE_EXIT {
PCHECK(setrlimit(RLIMIT_AS, &oldMemLimit) == 0);
};
#endif
TestExecutor executor(4);
// size of implicit promise is at least the size of the return.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment