Always use local activities for sync update based RPC

[longquanzheng]

Context
Today, whenever an RPC needs locking, we will use sync update of Temporal to process:

iwf/service/api/service.go

Line 620 in 20103a8

if needLocking(req) {

Within the sync update, we use an activity to invoke worker RPC API:

iwf/service/interpreter/workflowUpdater.go

Line 92 in 20103a8

err = u.provider.ExecuteActivity(&activityOutput, u.configer.ShouldOptimizeActivity(), ctx,

The activity will choose to use local activity or regular based on workflow config.

However, the config can be ignored and always use local activity, because there is no side effect of using local activity for this case, but only benefits. The latency will be lower with LA, and saving the storage cost becaues omitting the activity input in LA.

Goal
Update the workflowUpdate to always use local activity.

Note:

• Needs a versioning because this is not determinstic change