Mike Percy has submitted this change and it was merged.
Change subject: Remove InMemoryEnv completely
..
Remove InMemoryEnv completely
Every time we extend the Env interface we need to create a mock
implementation for MemEnv. M
Mike Percy has posted comments on this change.
Change subject: Remove InMemoryEnv completely
..
Patch Set 2: Verified+1
Overriding unrelated flaky test
--
To view, visit http://gerrit.cloudera.org:8080/3258
To unsubscribe, vi
Adar Dembo has posted comments on this change.
Change subject: Remove InMemoryEnv completely
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3258
To unsubscribe, visit http://gerrit.cloudera.org
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3258
to look at the new patch set (#2).
Change subject: Remove InMemoryEnv completely
..
Remove InMemoryEnv completely
Mike Percy has posted comments on this change.
Change subject: Remove InMemoryEnv completely
..
Patch Set 1:
Ah, should've used git grep.
--
To view, visit http://gerrit.cloudera.org:8080/3258
To unsubscribe, visit http://ger
Adar Dembo has posted comments on this change.
Change subject: Remove InMemoryEnv completely
..
Patch Set 1:
> There are no mentions of MemEnv or InMemoryEnv anywhere else in the code base
> that I could find.
I found a coupl
Mike Percy has posted comments on this change.
Change subject: Remove InMemoryEnv completely
..
Patch Set 1:
There are no mentions of MemEnv or InMemoryEnv anywhere else in the code base
that I could find.
For the EnvWrapper,
Adar Dembo has posted comments on this change.
Change subject: Remove InMemoryEnv completely
..
Patch Set 1:
I think memenv would have been useful had we ever used it in non-trivial tests
(i.e. tests that perform real I/O). Bu
Hello Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3258
to review the following change.
Change subject: Remove InMemoryEnv completely
..
Remove InMemoryEnv completely
Every t
