chore: revert "chore: trim eventNames sent to reporting if length exc…

…eeds 50 characters" (#5155)
rudderlabs · Oct 1, 2024 · 17c5ad7 · 17c5ad7
1 parent 61e947c
commit 17c5ad7
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/config/config.yaml b/config/config.yaml
@@ -249,3 +249,5 @@ PgNotifier:
   retriggerCount: 500
   trackBatchInterval: 2s
   maxAttempt: 3
+Reporting:
+  eventNameMaxLength: 0
diff --git a/enterprise/reporting/reporting.go b/enterprise/reporting/reporting.go
@@ -649,6 +649,7 @@ func (r *DefaultReporter) Report(ctx context.Context, metrics []*types.PUReporte
 	}
 	defer func() { _ = stmt.Close() }()
 
+	eventNameMaxLength := config.GetInt("Reporting.eventNameMaxLength", 0)
 	reportedAt := time.Now().UTC().Unix() / 60
 	for _, metric := range metrics {
 		workspaceID := r.configSubscriber.WorkspaceIDFromSource(metric.ConnectionDetails.SourceID)
@@ -662,9 +663,8 @@ func (r *DefaultReporter) Report(ctx context.Context, metrics []*types.PUReporte
 			metric = transformMetricForPII(metric, getPIIColumnsToExclude())
 		}
 
-		eventName := metric.StatusDetail.EventName
-		if len(eventName) > 50 {
-			metric.StatusDetail.EventName = fmt.Sprintf("%s...%s", eventName[:40], eventName[len(eventName)-10:])
+		if eventNameMaxLength > 0 && len(metric.StatusDetail.EventName) > eventNameMaxLength {
+			metric.StatusDetail.EventName = types.MaxLengthExceeded
 		}
 
 		_, err = stmt.Exec(

diff --git a/utils/types/reporting_types.go b/utils/types/reporting_types.go
@@ -21,6 +21,8 @@ const (
 	DefaultReplayEnabled    = false
 )
 
+const MaxLengthExceeded = ":max-length-exceeded:"
+
 const (
 	DiffStatus = "diff"
-Original file line number
+Diff line change
@@ Expand Up / @@ -21,6 +21,8 @@ const ( @@
     	DefaultReplayEnabled    = false
     )
+    const MaxLengthExceeded = ":max-length-exceeded:"
     const (
     	DiffStatus = "diff"
@@ Expand Down @@