Thank you for making this fix! I tested with the other support issue we had and it does fix it as well! Just added a note to also fix some of the zone config serialization while you're at it.

Run eslint and make sure Goland is configured to autoformat w/ it.

I'd add a second release note for db console. My understanding is that it's fine to have multiple release note entries if your PR covers multiple categories.

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

pkg/ui/src/views/shared/components/sql/highlight.tsx, line 65 at r1 (raw file):

        <br />
        <span className="hljs-label">constraints = ['</span>
        <span className="hljs-built_in">{String(zoneConfig.constraints)}</span>

Could we update this to print the proper constraints instead of constraints = ['[object Object],[object Object]'],. There are only two fields here and it should look like:

demo@127.0.0.1:26257/movr> SHOW ZONE CONFIGURATION FOR TABLE users;
    target    |                           raw_config_sql
--------------+---------------------------------------------------------------------
  TABLE users | ALTER TABLE users CONFIGURE ZONE USING
              |     range_min_bytes = 16777216,
              |     range_max_bytes = 67108864,
              |     gc.ttlseconds = 9000,
              |     num_replicas = 3,
              |     constraints = '{+region=us-east1: 1, +region=us-west1: 1}',
              |     lease_preferences = '[[+region=us-east1], [+region=us-west1]]'
(1 row)

pkg/ui/src/views/shared/components/sql/highlight.tsx, line 70 at r1 (raw file):

        <span className="hljs-label">lease_preferences = [['</span>
        <span className="hljs-built_in">
          {String(zoneConfig.lease_preferences)}

Same comment here as above, would be nice to get the serialization bug out of the way as well here. Example up above.

thanks for doing this!

Reviewed 6 of 8 files at r2.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @dhartunian and @nkodali)

pkg/server/admin.go, line 612 at r2 (raw file):

		const rawConfigSqlColName = "raw_config_sql"
		if len(rows) != 1 {
			return nil, s.serverErrorf("show zone configuration not available.")

Is it possible that we could have a perfectly valid table with an empty zone config?

TFTR. Updated to more gracefully handle empty zone config.

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

pkg/server/admin.go, line 612 at r2 (raw file):

My limited understanding of zone configs is that there is always a default. Despite that, I think it's safer to return empty string in the case this query gives no results, rather than a server error. Updated to do that.

pkg/ui/src/views/shared/components/sql/highlight.tsx, line 65 at r1 (raw file):

Done.

pkg/ui/src/views/shared/components/sql/highlight.tsx, line 70 at r1 (raw file):

Done.