Skip BufferedEvents persistence when empty (#707)

shawnhathaway · web-flow · commit d95406a5ccca · 2020-09-01T16:32:11.000-07:00
diff --git a/common/persistence/executionStore.go b/common/persistence/executionStore.go
@@ -406,7 +406,7 @@ func (m *executionManagerImpl) SerializeWorkflowMutation(
 	}
 
 	var serializedNewBufferedEvents *serialization.DataBlob
-	if input.NewBufferedEvents != nil {
+	if len(input.NewBufferedEvents) > 0 {
 		serializedNewBufferedEvents, err = m.serializer.SerializeBatchEvents(input.NewBufferedEvents, encoding)
 		if err != nil {
 			return nil, err

Original file line number	Diff line number	Diff line change
`@@ -406,7 +406,7 @@ func (m *executionManagerImpl) SerializeWorkflowMutation(`
`406`	`406`	`}`
`407`	`407`
`408`	`408`	`var serializedNewBufferedEvents *serialization.DataBlob`
`409`		`- if input.NewBufferedEvents != nil {`
	`409`	`+ if len(input.NewBufferedEvents) > 0 {`
`410`	`410`	`serializedNewBufferedEvents, err = m.serializer.SerializeBatchEvents(input.NewBufferedEvents, encoding)`
`411`	`411`	`if err != nil {`
`412`	`412`	`return nil, err`