Tim Deeb-SwihartandDavid Reiss 49dffe7ffa sql: Only update last refresh time when we refresh (#6538) ...

Fixes #6514 

## What changed?
I fixed a bug in our sql `DBHandle` that could cause us to never
reconnect to the database when an issue occurs.

## Why?
We should only update the last refresh time when we actually refresh. As
written we always update the last refresh time even when we don't
refresh.

If _every_ attempt to reconnect happens within
`sessionRefreshMinInterval` of the last attempt, the server will _never_
actually reconnect as it will just keep bumping the last refresh...

## How did you test it?
Existing tests

## Potential risks
None

## Documentation
No changes

## Is hotfix candidate?
Absolutely (cc @yiminc)

2024-10-04 14:12:44 -07:00

..

sqlplugin

sql: Only update last refresh time when we refresh (#6538)

2024-10-04 14:12:44 -07:00

cluster_metadata.go

Persistence context Part 7: SQL store implementation (#2711)

2022-04-13 10:08:07 -07:00

common.go

Nexus accumulated work (#5225)

2024-03-05 22:37:21 +00:00

errors.go

Use start time to dedup executions start (#6360)

2024-08-01 10:26:30 -07:00

execution_state_map.go

Inline UpdateInfo Storage (#4420)

2023-05-31 13:46:55 -04:00

execution_state_non_map.go

Use serviceerror.Unavailable error type instead of serviceerror.Internal (#1916)

2021-09-13 18:31:14 -07:00

execution_tasks.go

Remove history queue reader registration logic (#5420)

2024-02-20 14:44:38 -08:00

execution_util.go

Add start_time column to the current execution table. Some refactoring (#6310)

2024-07-24 14:17:46 -07:00

execution.go

populate workflow execution start time (#6354)

2024-07-30 15:25:40 -07:00

factory.go

Reconnect to SQL databases when connections fail (#5926)

2024-05-17 16:59:15 -07:00

history_store.go

Nexus accumulated work (#5225)

2024-03-05 22:37:21 +00:00

metadata.go

Add NamespaceInvalidState and NamespaceNotFound errors (#2785)

2022-04-30 00:03:47 -07:00

nexus_endpoint_store.go

Use namespace ID when persisting Nexus Endpoints (#6006)

2024-05-28 17:04:45 -07:00

queue_v2.go

Replace gogo/protobuf with google/protobuf (#5032)

2023-11-21 15:19:37 -08:00

queue.go

Replace gogo/protobuf with google/protobuf (#5032)

2023-11-21 15:19:37 -08:00

shard.go

Deprecate existing shard ownership assertion usage (#5974)

2024-05-31 12:27:48 -07:00

store.go

Replace golang.org/x/exp with stdlib where possible (#6366)

2024-08-01 19:33:04 +00:00

task.go

Merge versioning-2 branch into main

2024-04-12 13:30:59 -07:00

test_sql_persistence.go

Reconnect to SQL databases when connections fail (#5926)

2024-05-17 16:59:15 -07:00

version_checker.go

Reconnect to SQL databases when connections fail (#5926)

2024-05-17 16:59:15 -07:00