Skip to content
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

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

Jackie-Jiang
Copy link
Contributor

To better reflect the correct usage of each config

@Jackie-Jiang Jackie-Jiang added the user-experience Related to user experience label Sep 21, 2023
@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2023

Codecov Report

Merging #11652 (df58257) into master (3dce808) will decrease coverage by 0.05%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@             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     
Flag Coverage Δ
integration <0.01% <ø> (ø)
integration1 <0.01% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.07% <ø> (-0.01%) ⬇️
java-17 62.95% <ø> (-0.05%) ⬇️
java-20 62.96% <ø> (+<0.01%) ⬆️
temurin 63.09% <ø> (-0.05%) ⬇️
unittests 63.08% <ø> (-0.05%) ⬇️
unittests1 67.31% <ø> (-0.03%) ⬇️
unittests2 14.45% <ø> (-0.03%) ⬇️

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

Copy link
Contributor

@walterddr walterddr left a 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"
}
]
Copy link
Contributor

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?

Copy link
Contributor Author

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

@Jackie-Jiang Jackie-Jiang merged commit be9b7d6 into apache:master Sep 22, 2023
21 checks passed
@Jackie-Jiang Jackie-Jiang deleted the quick_start_table_config branch September 22, 2023 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup user-experience Related to user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants