-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update the table config in quick start #11652
Update the table config in quick start #11652
Conversation
Codecov Report
@@ Coverage Diff @@
## master #11652 +/- ##
============================================
- Coverage 63.13% 63.09% -0.05%
+ Complexity 1121 1120 -1
============================================
Files 2335 2335
Lines 125216 125216
Branches 19209 19209
============================================
- Hits 79054 79003 -51
- Misses 40526 40580 +54
+ Partials 5636 5633 -3
Flags with carried forward coverage won't be shown. Click here to find out more. see 15 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
lgtm
"realtime.segment.flush.threshold.time": "12h", | ||
"realtime.segment.flush.threshold.size": "100000" | ||
} | ||
] |
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.
any reason why the kafka version doesn't have the derived transformConfigs columns?
either make the consistent or keep a "json" name in the filename like the upsert side table config?
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.
Good catch. I think I made the wrong copy paste
31caa0f
to
df58257
Compare
To better reflect the correct usage of each config