sql: skip pgbench if not on PATH

[dt]

Currently blocking #8081

This change is

[tamird]

please mention the issue.

---

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks pending.

---

pkg/sql/pgbench_test.go, line 24 at r1 (raw file):

  "math/rand"
  "net/url"
  osexec "os/exec"

why the alias?

---

Comments from Reviewable

[dt]

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks pending.

---

pkg/sql/pgbench_test.go, line 24 at r1 (raw file):

Previously, tamird (Tamir Duberstein) wrote…

why the alias?

there's already a symbol `exec` in the sql package (in txn_restart_test.go).

---

Comments from Reviewable

[tamird]

Review status: all files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.

---

pkg/sql/pgbench_test.go, line 24 at r1 (raw file):

Previously, dt (David Taylor) wrote…

there's already a symbol exec in the sql package (in txn_restart_test.go).

which has one caller 🤦

---

Comments from Reviewable

[jordanlewis]

---

Review status: all files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.

---

Comments from Reviewable

[dt]

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion.

---

pkg/sql/pgbench_test.go, line 24 at r1 (raw file):

Previously, tamird (Tamir Duberstein) wrote…

which has one caller 🤦

Done.

---

Comments from Reviewable

[tamird]

Can you mention the issue in the commit message, not just the PR description?

---

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.

---

Comments from Reviewable

[dt]

Done.

[tamird]

Great. Feel free to unassign yourself from #8081 and kick off a benchmark run when this merges.