storage: add CommitNoSyncWait and SyncWait to Batch interface

[nvanbenschoten]

Extracted from #94165.

Picks up cockroachdb/pebble/pull/2117.

Release note: None
Epic: None

[blathers-crl]

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

_{🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.}

[cockroach-teamcity]

This change is 

[sumeerbhola]

Reviewed 3 of 3 files at r1.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @nvanbenschoten)

---

pkg/storage/pebble_batch.go line 559 at r1 (raw file):

		// Once we do that, document on the storage.Batch interface the meaning of
		// an error returned from this method and the guarantees that callers have
		// or don't have after they receive and error from this method.

nit: an eror ...

---

pkg/storage/pebble_batch.go line 578 at r1 (raw file):

		// Once we do that, document on the storage.Batch interface the meaning of
		// an error returned from this method and the guarantees that callers have
		// or don't have after they receive and error from this method.

ditto

[nvanbenschoten]

TFTR!

bors r=sumeerbhola

Reviewable status: complete! 0 of 0 LGTMs obtained (and 1 stale) (waiting on @sumeerbhola)

---

pkg/storage/pebble_batch.go line 559 at r1 (raw file):

Previously, sumeerbhola wrote…

nit: an eror ...

Done.

---

pkg/storage/pebble_batch.go line 578 at r1 (raw file):

Previously, sumeerbhola wrote…

ditto

Done.

[craig]

Build succeeded:

• Bazel Essential CI (Cockroach)