-
Notifications
You must be signed in to change notification settings - Fork 297
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.
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
chore: added trino and spark test #3525
Conversation
fe7f3d3
to
6293162
Compare
6293162
to
4ff9587
Compare
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #3525 +/- ##
==========================================
+ Coverage 68.05% 68.12% +0.06%
==========================================
Files 318 318
Lines 50382 50382
==========================================
+ Hits 34289 34324 +35
+ Misses 13867 13835 -32
+ Partials 2226 2223 -3 ☔ View full report in Codecov by Sentry. |
0df87f7
to
76b62a9
Compare
76b62a9
to
1f6da05
Compare
f0bb66d
to
843ba1a
Compare
7e88b67
to
ebe94c3
Compare
ebe94c3
to
ac88374
Compare
require.Eventually(t, func() bool { | ||
_, err := db.ExecContext(ctx, "SELECT 1") | ||
return err == nil | ||
}, 60*time.Second, 1*time.Second) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could try more often, we don't even have log pollution here. Every 100ms for example could work.
|
||
var count int64 | ||
|
||
require.NoError(t, testhelper.WithConstantRetries(func() error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest using require.Eventually
with a short interval here as well, for sure one that is a fraction of a second. Same applies for others below.
26a6247
to
fc1a3fa
Compare
fc1a3fa
to
c8d815b
Compare
Description
Notion Ticket
https://www.notion.so/rudderstacks/Investigate-parquet-issue-for-AWS-GLUE-Presto-trino-6c8280c81d3a47eb9d54138586ab44d0?pvs=4
Security