New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[internal/otelarrow] Fix test flake (for 34719) #34889

Merged

codeboten merged 19 commits into open-telemetry:main from jmacd:jmacd/fix34719

Sep 6, 2024

remove 1 line

Loading branch information

jmacd committed Aug 27, 2024

commit de29f5262623386a7a61c607921cc2d9da608863

internal/otelarrow/test/e2e_test.go

-Original file line number
+Diff line change
@@ Expand Up / @@ -434,8 +434,7 @@ func TestIntegrationMemoryLimited(t *testing.T) { @@
     					cnt++
     				}
     			}
-    			ok := cnt == 0 || test.sentSpans.Load() < 100
-    			return ok
+    			return cnt == 0 || test.sentSpans.Load() < 100
     		},
     	}
@@ Expand Down @@