Closed petermattis closed 6 years ago
TestContinuousLoad_BlockWriter
does the same thing as the two kv
tests in Workload Nightly, so I think we can check that one. TestContinuousLoad_Photos should be easy to add once we have a workload impl of photos
Ported BenchmarkRestore2TB
in https://github.com/cockroachdb/cockroach/pull/22859 (forgot to link this issue there). I don't think either of BenchmarkRestoreBig
or BenchmarkRestoreTPCH10
are really pulling their weight, so I'm going to skip both of them unless someone objects
No objections. I say delete them outright! 🔨
On Tue, Feb 20, 2018 at 4:51 PM, Daniel Harrison notifications@github.com wrote:
Ported BenchmarkRestore2TB in #22859 https://github.com/cockroachdb/cockroach/pull/22859 (forgot to link this issue there). I don't think either of BenchmarkRestoreBig or BenchmarkRestoreTPCH10 are really pulling their weight, so I'm going to skip both of them unless someone objects
— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/cockroachdb/cockroach/issues/22646#issuecomment-367132318, or mute the thread https://github.com/notifications/unsubscribe-auth/AA15IL9rfyzHwZEbBJxjCCi2a3zgS4Pjks5tWz5GgaJpZM4SEG1b .
Do we want to bother porting photos
to workload
? TPC-C seems like a better workload.
Agreed. I'll skip that one too
Okay, the only one left is the schema change one. @vivekmenezes just to confirm, do we think that's still useful and worth porting?
I'm not familiar with the details of the existing schema change test, but we definitely want a nightly schema change test.
@danhhz yes that's a super useful test! Thanks
@danhhz it appears you have done all the tasks for this issue, save closing it. Can you please confirm and close?
I think there's one final task: removing the old Terraform tests. Dan did all the hard work here so I don't want to spoil his fun. Punting out of 2.0 in the meantime, though.
Tracking issue for porting terrafarm-based nightly tests to
roachtest
.