Files
Stephan BehnkeandGitHub c01b3b105a Remove Put cache method (#6862)
## What changed?
<!-- Describe what has changed in this PR -->

Remove `Put` method from workflow cache.

## Why?
<!-- Tell your future self why have you made these changes -->

Not needed anymore after
https://github.com/temporalio/temporal/pull/6850.

## How did you test it?
<!-- How have you verified this change? Tested locally? Added a unit
test? Checked in staging env? -->

## Potential risks
<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->

## Documentation
<!-- Have you made sure this change doesn't falsify anything currently
stated in `docs/`? If significant
new behavior is added, have you described that in `docs/`? -->

## Is hotfix candidate?
<!-- Is this PR a hotfix candidate or does it require a notification to
be sent to the broader community? (Yes/No) -->
2024-11-22 10:53:49 -08:00

503 lines
14 KiB
Go

// The MIT License
//
// Copyright (c) 2020 Temporal Technologies Inc. All rights reserved.
//
// Copyright (c) 2020 Uber Technologies, Inc.
//
// Permission is hereby granted, free of charge, to any person obtaining a copy
// of this software and associated documentation files (the "Software"), to deal
// in the Software without restriction, including without limitation the rights
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
// copies of the Software, and to permit persons to whom the Software is
// furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included in
// all copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
//go:generate mockgen -copyright_file ../../../../LICENSE -package $GOPACKAGE -source $GOFILE -destination cache_mock.go
package cache
import (
"context"
"sync/atomic"
"time"
"unicode/utf8"
"github.com/pborman/uuid"
commonpb "go.temporal.io/api/common/v1"
"go.temporal.io/api/serviceerror"
"go.temporal.io/server/common/cache"
"go.temporal.io/server/common/definition"
"go.temporal.io/server/common/finalizer"
"go.temporal.io/server/common/headers"
"go.temporal.io/server/common/locks"
"go.temporal.io/server/common/log"
"go.temporal.io/server/common/log/tag"
"go.temporal.io/server/common/metrics"
"go.temporal.io/server/common/namespace"
"go.temporal.io/server/common/persistence"
"go.temporal.io/server/service/history/configs"
"go.temporal.io/server/service/history/consts"
"go.temporal.io/server/service/history/shard"
"go.temporal.io/server/service/history/workflow"
)
type (
// ReleaseCacheFunc must be called to release the workflow context from the cache.
// Make sure not to access the mutable state or workflow context after releasing back to the cache.
// If there is any error when using the mutable state (e.g. mutable state is mutated and dirty), call release with
// the error so the in-memory copy will be thrown away.
ReleaseCacheFunc func(err error)
Cache interface {
GetOrCreateCurrentWorkflowExecution(
ctx context.Context,
shardContext shard.Context,
namespaceID namespace.ID,
workflowID string,
lockPriority locks.Priority,
) (ReleaseCacheFunc, error)
GetOrCreateWorkflowExecution(
ctx context.Context,
shardContext shard.Context,
namespaceID namespace.ID,
execution *commonpb.WorkflowExecution,
lockPriority locks.Priority,
) (workflow.Context, ReleaseCacheFunc, error)
}
cacheImpl struct {
cache.Cache
onPut func(wfContext *workflow.Context)
onEvict func(wfContext *workflow.Context)
nonUserContextLockTimeout time.Duration
}
cacheItem struct {
shardId int32
wfContext workflow.Context
finalizer *finalizer.Finalizer
}
NewCacheFn func(config *configs.Config, logger log.Logger, handler metrics.Handler) Cache
Key struct {
// Those are exported because some unit tests uses the cache directly.
// TODO: Update the unit tests and make those fields private.
WorkflowKey definition.WorkflowKey
ShardUUID string
}
)
var NoopReleaseFn ReleaseCacheFunc = func(err error) {}
const (
cacheNotReleased int32 = 0
cacheReleased int32 = 1
)
const (
workflowLockTimeoutTailTime = 500 * time.Millisecond
)
func NewHostLevelCache(
config *configs.Config,
logger log.Logger,
handler metrics.Handler,
) Cache {
maxSize := config.HistoryHostLevelCacheMaxSize()
if config.HistoryCacheLimitSizeBased {
maxSize = config.HistoryHostLevelCacheMaxSizeBytes()
}
return newCache(
maxSize,
config.HistoryCacheTTL(),
config.HistoryCacheNonUserContextLockTimeout(),
logger,
handler,
)
}
func NewShardLevelCache(
config *configs.Config,
logger log.Logger,
handler metrics.Handler,
) Cache {
maxSize := config.HistoryShardLevelCacheMaxSize()
if config.HistoryCacheLimitSizeBased {
maxSize = config.HistoryShardLevelCacheMaxSizeBytes()
}
return newCache(
maxSize,
config.HistoryCacheTTL(),
config.HistoryCacheNonUserContextLockTimeout(),
logger,
handler,
)
}
func newCache(
size int,
ttl time.Duration,
nonUserContextLockTimeout time.Duration,
logger log.Logger,
handler metrics.Handler,
) Cache {
opts := &cache.Options{
TTL: ttl,
Pin: true,
OnPut: func(val any) {
//revive:disable-next-line:unchecked-type-assertion
item := val.(*cacheItem)
if item.finalizer == nil {
return // should only happen in unit tests
}
wfKey := item.wfContext.GetWorkflowKey()
err := item.finalizer.Register(wfKey.String(), func(ctx context.Context) error {
if err := item.wfContext.Lock(ctx, locks.PriorityHigh); err != nil {
return err
}
defer item.wfContext.Unlock()
item.wfContext.Clear()
return nil
})
if err != nil {
logger.Debug("cache failed to register callback in finalizer",
tag.Error(err), tag.ShardID(item.shardId))
}
},
OnEvict: func(val any) {
//revive:disable-next-line:unchecked-type-assertion
item := val.(*cacheItem)
if item.finalizer == nil {
return // should only happen in unit tests
}
wfKey := item.wfContext.GetWorkflowKey()
err := item.finalizer.Deregister(wfKey.String())
if err != nil {
// debug level since this is very common: the cache item was registered with a finalizer
// that has been finalized since then and is therefore no longer accepting any calls
logger.Debug("cache failed to de-register callback in finalizer",
tag.Error(err), tag.ShardID(item.shardId))
}
},
}
withMetrics := cache.NewWithMetrics(size, opts, handler.WithTags(metrics.CacheTypeTag(metrics.MutableStateCacheTypeTagValue)))
return &cacheImpl{
Cache: withMetrics,
nonUserContextLockTimeout: nonUserContextLockTimeout,
}
}
func (c *cacheImpl) GetOrCreateCurrentWorkflowExecution(
ctx context.Context,
shardContext shard.Context,
namespaceID namespace.ID,
workflowID string,
lockPriority locks.Priority,
) (ReleaseCacheFunc, error) {
if err := c.validateWorkflowID(workflowID); err != nil {
return nil, err
}
handler := shardContext.GetMetricsHandler().WithTags(
metrics.OperationTag(metrics.HistoryCacheGetOrCreateCurrentScope),
metrics.CacheTypeTag(metrics.MutableStateCacheTypeTagValue),
metrics.NamespaceIDTag(namespaceID.String()),
)
metrics.CacheRequests.With(handler).Record(1)
start := time.Now()
defer func() { metrics.CacheLatency.With(handler).Record(time.Since(start)) }()
execution := commonpb.WorkflowExecution{
WorkflowId: workflowID,
// using empty run ID as current workflow run ID
RunId: "",
}
_, weReleaseFn, err := c.getOrCreateWorkflowExecutionInternal(
ctx,
shardContext,
namespaceID,
&execution,
handler,
true,
lockPriority,
)
metrics.ContextCounterAdd(ctx, metrics.HistoryWorkflowExecutionCacheLatency.Name(),
time.Since(start).Nanoseconds())
return weReleaseFn, err
}
func (c *cacheImpl) GetOrCreateWorkflowExecution(
ctx context.Context,
shardContext shard.Context,
namespaceID namespace.ID,
execution *commonpb.WorkflowExecution,
lockPriority locks.Priority,
) (workflow.Context, ReleaseCacheFunc, error) {
if err := c.validateWorkflowExecutionInfo(ctx, shardContext, namespaceID, execution, lockPriority); err != nil {
return nil, nil, err
}
handler := shardContext.GetMetricsHandler().WithTags(
metrics.OperationTag(metrics.HistoryCacheGetOrCreateScope),
metrics.CacheTypeTag(metrics.MutableStateCacheTypeTagValue),
metrics.NamespaceIDTag(namespaceID.String()),
)
metrics.CacheRequests.With(handler).Record(1)
start := time.Now()
defer func() { metrics.CacheLatency.With(handler).Record(time.Since(start)) }()
weCtx, weReleaseFunc, err := c.getOrCreateWorkflowExecutionInternal(
ctx,
shardContext,
namespaceID,
execution,
handler,
false,
lockPriority,
)
metrics.ContextCounterAdd(ctx, metrics.HistoryWorkflowExecutionCacheLatency.Name(),
time.Since(start).Nanoseconds())
return weCtx, weReleaseFunc, err
}
func (c *cacheImpl) getOrCreateWorkflowExecutionInternal(
ctx context.Context,
shardContext shard.Context,
namespaceID namespace.ID,
execution *commonpb.WorkflowExecution,
handler metrics.Handler,
forceClearContext bool,
lockPriority locks.Priority,
) (workflow.Context, ReleaseCacheFunc, error) {
cacheKey := Key{
WorkflowKey: definition.NewWorkflowKey(namespaceID.String(), execution.GetWorkflowId(), execution.GetRunId()),
ShardUUID: shardContext.GetOwner(),
}
item, cacheHit := c.Get(cacheKey).(*cacheItem)
var workflowCtx workflow.Context
if cacheHit {
workflowCtx = item.wfContext
} else {
metrics.CacheMissCounter.With(handler).Record(1)
workflowCtx = workflow.NewContext(
shardContext.GetConfig(),
cacheKey.WorkflowKey,
shardContext.GetLogger(),
shardContext.GetThrottledLogger(),
shardContext.GetMetricsHandler(),
)
var err error
value := &cacheItem{shardId: shardContext.GetShardID(), wfContext: workflowCtx, finalizer: shardContext.GetFinalizer()}
existing, err := c.PutIfNotExist(cacheKey, value)
if err != nil {
metrics.CacheFailures.With(handler).Record(1)
return nil, nil, err
}
//nolint:revive
workflowCtx = existing.(*cacheItem).wfContext
}
if err := c.lockWorkflowExecution(ctx, workflowCtx, cacheKey, lockPriority); err != nil {
metrics.CacheFailures.With(handler).Record(1)
metrics.AcquireLockFailedCounter.With(handler).Record(1)
return nil, nil, err
}
// TODO This will create a closure on every request.
// Consider revisiting this if it causes too much GC activity
releaseFunc := c.makeReleaseFunc(cacheKey, shardContext, workflowCtx, forceClearContext, handler, time.Now())
return workflowCtx, releaseFunc, nil
}
func (c *cacheImpl) lockWorkflowExecution(
ctx context.Context,
workflowCtx workflow.Context,
cacheKey Key,
lockPriority locks.Priority,
) error {
// skip if there is no deadline
if deadline, ok := ctx.Deadline(); ok {
var cancel context.CancelFunc
if headers.GetCallerInfo(ctx).CallerType != headers.CallerTypeAPI {
newDeadline := time.Now().Add(c.nonUserContextLockTimeout)
if newDeadline.Before(deadline) {
ctx, cancel = context.WithDeadline(ctx, newDeadline)
defer cancel()
}
} else {
newDeadline := deadline.Add(-workflowLockTimeoutTailTime)
if newDeadline.After(time.Now()) {
ctx, cancel = context.WithDeadline(ctx, newDeadline)
defer cancel()
}
}
}
if err := workflowCtx.Lock(ctx, lockPriority); err != nil {
// ctx is done before lock can be acquired
c.Release(cacheKey)
return consts.ErrResourceExhaustedBusyWorkflow
}
return nil
}
func (c *cacheImpl) makeReleaseFunc(
cacheKey Key,
shardContext shard.Context,
context workflow.Context,
forceClearContext bool,
handler metrics.Handler,
acquireTime time.Time,
) func(error) {
status := cacheNotReleased
return func(err error) {
if atomic.CompareAndSwapInt32(&status, cacheNotReleased, cacheReleased) {
defer func() {
metrics.HistoryWorkflowExecutionCacheLockHoldDuration.With(handler).Record(time.Since(acquireTime))
}()
if rec := recover(); rec != nil {
context.Clear()
context.Unlock()
c.Release(cacheKey)
panic(rec)
} else {
if err != nil || forceClearContext {
// TODO see issue #668, there are certain type or errors which can bypass the clear
context.Clear()
context.Unlock()
c.Release(cacheKey)
} else {
isDirty := context.IsDirty()
if isDirty {
context.Clear()
logger := log.With(shardContext.GetLogger(), tag.ComponentHistoryCache)
logger.Error("Cache encountered dirty mutable state transaction",
tag.WorkflowNamespaceID(context.GetWorkflowKey().NamespaceID),
tag.WorkflowID(context.GetWorkflowKey().WorkflowID),
tag.WorkflowRunID(context.GetWorkflowKey().RunID),
)
}
context.Unlock()
c.Release(cacheKey)
if isDirty {
panic("Cache encountered dirty mutable state transaction")
}
}
}
}
}
}
func (c *cacheImpl) validateWorkflowExecutionInfo(
ctx context.Context,
shardContext shard.Context,
namespaceID namespace.ID,
execution *commonpb.WorkflowExecution,
lockPriority locks.Priority,
) error {
if err := c.validateWorkflowID(execution.GetWorkflowId()); err != nil {
return err
}
// RunID is not provided, lets try to retrieve the RunID for current active execution
if execution.GetRunId() == "" {
runID, err := GetCurrentRunID(
ctx,
shardContext,
c,
namespaceID.String(),
execution.GetWorkflowId(),
lockPriority,
)
if err != nil {
return err
}
execution.RunId = runID
} else if uuid.Parse(execution.GetRunId()) == nil { // immediately return if invalid runID
return serviceerror.NewInvalidArgument("RunId is not valid UUID.")
}
return nil
}
func (c *cacheImpl) validateWorkflowID(
workflowID string,
) error {
if workflowID == "" {
return serviceerror.NewInvalidArgument("Can't load workflow execution. WorkflowId not set.")
}
if !utf8.ValidString(workflowID) {
// We know workflow cannot exist with invalid utf8 string as WorkflowID.
return serviceerror.NewNotFound("Workflow not exists.")
}
return nil
}
func GetCurrentRunID(
ctx context.Context,
shardContext shard.Context,
workflowCache Cache,
namespaceID string,
workflowID string,
lockPriority locks.Priority,
) (runID string, retErr error) {
currentRelease, err := workflowCache.GetOrCreateCurrentWorkflowExecution(
ctx,
shardContext,
namespace.ID(namespaceID),
workflowID,
lockPriority,
)
if err != nil {
return "", err
}
defer func() { currentRelease(retErr) }()
resp, err := shardContext.GetCurrentExecution(
ctx,
&persistence.GetCurrentExecutionRequest{
ShardID: shardContext.GetShardID(),
NamespaceID: namespaceID,
WorkflowID: workflowID,
},
)
if err != nil {
return "", err
}
return resp.RunID, nil
}
func (c *cacheItem) CacheSize() int {
if sg, ok := c.wfContext.(cache.SizeGetter); ok {
return sg.CacheSize()
}
return 0
}