Skip to content

Commit a8bdea6

Browse files
committed
fix: let's make this 10 MBs and the arbitary limits more than 5 MBs. most executions shouldn't be larger than this
1 parent f513355 commit a8bdea6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

shared.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17629,7 +17629,7 @@ func setExecutionVariable(actionResult ActionResult) bool {
1762917629

1763017630
// Finds execution results and parameters that are too large to manage and reduces them / saves data partly
1763117631
func compressExecution(ctx context.Context, workflowExecution WorkflowExecution, saveLocationInfo string) (WorkflowExecution, bool) {
17632-
if project.Environment != "cloud" {
17632+
if project.Environment == "worker" {
1763317633
log.Printf("[DEBUG][%s] No need to make this execution any smaller", workflowExecution.ExecutionId)
1763417634
return workflowExecution, false
1763517635
}
@@ -17797,7 +17797,7 @@ func compressExecution(ctx context.Context, workflowExecution WorkflowExecution,
1779717797
} else {
1779817798
// OpenSearch (on-premise) handling
1779917799
// log.Printf("[DEBUG] Result length is %d for execution Id %s, %s", len(tmpJson), workflowExecution.ExecutionId, saveLocationInfo)
17800-
if len(tmpJson) >= 1000000 {
17800+
if len(tmpJson) >= 10000000 {
1780117801
// Clean up results' actions
1780217802

1780317803
log.Printf("[DEBUG][%s](%s) ExecutionVariables size: %d, Result size: %d, executionArgument size: %d, Results size: %d", workflowExecution.ExecutionId, saveLocationInfo, len(workflowExecution.ExecutionVariables), len(workflowExecution.Result), len(workflowExecution.ExecutionArgument), len(workflowExecution.Results))
@@ -17807,7 +17807,7 @@ func compressExecution(ctx context.Context, workflowExecution WorkflowExecution,
1780717807
actionId := "execution_argument"
1780817808

1780917809
// Arbitrary reduction size
17810-
maxSize := 50000
17810+
maxSize := 5000000
1781117811
basepath := os.Getenv("SHUFFLE_FILE_LOCATION")
1781217812
if len(basepath) == 0 {
1781317813
basepath = "files"

0 commit comments

Comments
 (0)