-
Notifications
You must be signed in to change notification settings - Fork 894
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
[MD] Support local cluster by default #5811
[MD] Support local cluster by default #5811
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5811 +/- ##
==========================================
+ Coverage 66.78% 66.92% +0.13%
==========================================
Files 2591 3302 +711
Lines 48749 63505 +14756
Branches 8765 10140 +1375
==========================================
+ Hits 32557 42500 +9943
- Misses 14054 18540 +4486
- Partials 2138 2465 +327
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thanks for the comment. This change is to enabled local cluster by default when "data_source.enabled = true" and "data_source.defaultCluster" is commented out. I've update the related info under this task #5696 and we can use it as the meta issue. Thanks~ |
Signed-off-by: Xinrui Bai <xinruiba@amazon.com>
Signed-off-by: Xinrui Bai <xinruiba@amazon.com>
Signed-off-by: Xinrui Bai <xinruiba@amazon.com>
cb61189
to
d2ce8a6
Compare
Description
When MD enabled, local cluster should be enabled by default
Issues Resolved
#5754
Screenshot
Testing the changes
Tested when
We still able to add sample data successfully in local cluster
Check List
yarn test:jest
yarn test:jest_integration