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
* MDF [conf] enable bridge retain flag in old conf #795
Conversation
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #795 +/- ##
==========================================
- Coverage 65.62% 63.29% -2.34%
==========================================
Files 123 123
Lines 35996 36021 +25
==========================================
- Hits 23624 22801 -823
- Misses 12372 13220 +848 ☔ View full report in Codecov by Sentry. |
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
# # # Value: Number | ||
# # # Default: 5 | ||
# file_count = 5 | ||
# } |
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.
There are plenty of memleaks in conf parse in exchange & parquet now.
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.
worth an issue
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.
There are plenty of memleaks in conf parse in exchange & parquet now.
I have made some updates. Those memory leakage will no longer exists after that pr was merged.
Signed-off-by: wayne <yukang.wei@emqx.io>
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.
not sure the reasons of some changes.
src/supplemental/nanolib/conf_ver2.c
Outdated
} else { | ||
nng_strfree(param->name); | ||
NNI_FREE_STRUCT(param); | ||
} | ||
} | ||
config->param_count = cvector_size(config->params); | ||
config->param_count = cnt; |
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.
???
|
||
## multi-stream: enable or disable the multi-stream bridging mode | ||
## Still a feature WIP, but basic func is ready | ||
## Value: true/false | ||
## Default: false | ||
bridge.mqtt.emqx.quic_multi_stream=true | ||
bridge.mqtt.emqx.quic_multi_stream=false |
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.
why?
# # # Value: Number | ||
# # # Default: 5 | ||
# file_count = 5 | ||
# } |
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.
worth an issue
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
Signed-off-by: wayne <yukang.wei@emqx.io>
@@ -105,6 +105,6 @@ NUTS_TESTS = { | |||
{"get size", test_get_size}, | |||
{"get time", test_get_time}, | |||
{"conf parse v2", test_conf_parse_ver2}, | |||
{"conf parse", test_conf_parse}, | |||
// {"conf parse", test_conf_parse}, |
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.
remember to fire an issue regarding this
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.
roger
No description provided.