mirror of
https://github.com/hanzoai/tasks.git
synced 2026-10-30 14:56:19 +00:00
## 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) -->
503 lines
14 KiB
Go
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
|
|
}
|