-
Notifications
You must be signed in to change notification settings - Fork 81
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
Dev/1.1.0 #785
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* feat: switch null functions to static inline * feat: switch check fct to static inline * feat: check if there are subs to trigger * feat: pass by ref with keyexpr alias and duplicate * feat: switch null fct to return {0} * feat: add local sub config token * feat: simplify slice_init / string_preallocate * feat: add expanded key case * feat: make _alias fct static inline * doc: fix typo * fix: add local subscriber token to config.h * doc: add warning on null functions * feat: add reply_data_init function
* feat: more static inline * feat: skip null string copy in encoding * feat: add no-weak refcount * feat: streamline vec_make * feat: add publisher check session config token * fix: simple rc memory leak * feat: switch arc_slice to simple rc * feat: add valid flag to timestamp * feat: add timestamp_move function * feat: switch sample_create to pass by reference * fix: missing token on publisher_delete * feat: update trigger subscription calls * fix: dummy sample_create prototype * feat: remove null timestamp value in trigger subs * feat: optimize timestamps cost * feat: check string before encoding move * feat: alias instead of copy payload on decode * doc: explicit read task errors * doc: rework transport/codec logs * feat: keep going on multicast message processing error * fix: revert payload aliasing * fix: set timestamp valid only if decode successful * fix: review comment
* feat: add refcount to z_buf * feat: wrap payload instead of copy * fix: segfaults on tests and examples * feat: pass arg by reference in reply_create * feat: add zbuf copy bytes * fix: zbuf clear memory leak * feat: improve read task buffer realloc logic * doc: update read task comment * feat: add raweth read task buffer allocation * feat: move rx buffer logic to function * fix: zbuf init code * feat: add rc count function
* fix: bad nt string access in msgcodec test * feat: wrap fragment instead of copy * feat: remove extra wbuf init allocation * feat: add iobuf null functions * feat: add wbuf moved as zbuf function * feat: add defrag buff state and allocate only as needed * feat: move defrag buffer instead of copy * fix: wbuf moved memory leak * fix: memory leak on write filter * feat: add defrag buffer changes to multicast * fix: transport memory leak
* feat: add _vec_steal and _vec_null * chore: remove stray ; * feat: add network message copy function * fix: memory leak * feat: abstract transport mutex * build: add batching config token * feat: add transport batching state functions * feat: add multicast batching * refactor: unicast tx function * feat: add unicast tx batching * feat: add batching api * fix: multicast peer mutex * fix: single thread compilation * feat: restore wbuf if batch if too full * fix: redundant buf reset * feat: pass sn by value * feat: send fragment with batching if needed * feat: rename batch api functions * feat: switch transport mutex functions to static inline * doc: add batching api doc * fix: boolean condition
* refactor: rename session mutex function * fix: api options recopy * feat: add local query timeout info * feat: add local lazy query timeout check * feat: centralize query timeout process
* feat: add tcp dummy functions * feat: add unix listen_tcp * fix: don't print error if connection is closed * feat: add is_open arg for peer mode * fix: listen tcp * feat: add peer unicast * refactor: _z_open * fix: peer multicast test config * fix: timing * feat: switch from bool to enum * feat: remove tcp peer timing
* feat: switch id empty/check to static inline * feat: use batch count instead of batch vec * refactor: batch don't copy messages * feat: rename batch api * feat: group common transport fields * feat: use common struct in tx functions * feat: rename multicast tx functions * fix: unreachable * refactor: fuse unicast and multicast tx
* feat: alias string decode * fix: alias keyexpr only on rx buffer * feat: add svec release function * feat: remove duplicate message clear * feat: remove unused subscription function * fix: missing const qualifier * feat: remove encoding null value * feat: improve sample create perf * feat: init string in encoding move * feat: replace rc sub list with sub info vec * feat: check before encoding clear * feat: avoid rechecking msg reliability * feat: add sub memoization * fix: alias cache if suffix * feat: move z_bytes instead of copy * feat: add svec move function * feat: add memoization config token * feat: add vec alias function * feat: switch sub cache to session level * feat: add svec expand function * fix: missing function args * fix: don't use _z_noop_move in svec * feat: svec ops return error instead of bool * feat: svec functions * feat: add use elem f in svec * feat: switch frame nmsg to svec * feat: add config value for initial frame size evaluation * feat: remove superfluous init * fix: rename cache token * fix: remove superfluous initialization
* fix: remove superfluous rc init * feat: lazify svec release * refactor: rename slice_empty as slice_null * feat: rework svec expand * refactor: z_bytes_append_slice * feat: lazify arc slice * feat: improve stirng/slice move * feat: lazify sample timestamp set * feat: add non-reader decode functions * feat: remove superfluous keyexpr clear * feat: add bytes alias arc * feat: add rx pool size config option * feat: use transport arc slice pool for payload decode * feat: n msg svec is now a transport resource pool * fix: add offset to svec init * feat: make svec use elem f a per call arg * feat: skip ke suffix check * fix: arc pool * feat: improve sub perf and readability * feat: add z_string_alias_slice function * feat: align queryable with subscription * feat: align reply with sub & queryables * feat: nothing to clear in frame * fix: reply clean up * feat: streamline replies * feat: query is not a rc and store a session rc * fix: ci issues * fix: attachment examples * fix: keyexpr equals * fix: flaky test
* Update closure handlers documentaion (#736) * Rename closure callback/dropper types (#739) * Remove legacy z_zint_t from public API (#741) * Add `export "C"` for all header files (#740) * Add `export "C"` for proper closures callbacks export * Add `export "C"` for all headers * add serialize from substr functions (#742) * Rename z_loaned_hello_handler_t to z_closure_hello_callback_t (#743) * Mark z_reply_replier_id as unstable (#745) * Add documentation about logging (#744) * Update README.md (#746) * hardcode clang-format runner to Ubuntu24.04 (#748) * fix build granularity (#747) * fix: badly named constant (#750) * fix packages * fix: update debian packaging - include libzenohpico.so in debian package - change the package name to libzenohpico in line with libzenohc - set correct debian version for pre releases - fix wrong version used in Release mode * fix: debian dev package name * fix: package version for releases (#753) * fix: debian_version for official releases otherwise they would be undefined * fix: align cpack version with zenoh-c * chore: review comments move cpack version closer to where it's used * build shared lib for packages instead of static one (#757) * Fix z_task cleanup for platforms with pthread support (#759) * Fix read/lease task cleanup (#760) * Align ID string representation with zenoh (lowercase) (#761) * Remove default parameters from z_closure (#755) * build both libraries:shared and static when packaging; (#766) add static library to dev package; * Publish debian packages (#769) * fix: add workflow to release debian packages * fix: align with zenoh-c - update package names - set version string in the same way - set DEBARCH/RPMARCH - set CPACK_PACKAGE_FILE_NAME - add -j to zip archive to not include parent folder * fix: use CPACK_PACKAGE_NAME * fix: pass package name to all build targets * fix: Don't tag release branch during dry-run * Replace exit with return in zephyr examples (#774) * Add platform_common.c to zephyr CMakeLists.txt file * Rework Zenoh ID conversion * chore: Update org secrets (#782) As per eclipse-zenoh/.eclipsefdn#18, secrets were updated to follow eclipse foundation naming convention. * fix: rebase shenanigans * feat: set batching api as unstable --------- Co-authored-by: Alexander Bushnev <Alexander@Bushnev.pro> Co-authored-by: DenisBiryukov91 <155981813+DenisBiryukov91@users.noreply.github.com> Co-authored-by: Denis Biryukov <denis.biryukov@zettascale.tech> Co-authored-by: Diogo Mendes Matsubara <diogo.matsubara@zettascale.tech> Co-authored-by: Luca Cominardi <luca.cominardi@gmail.com>
PR missing one of the required labels: {'documentation', 'dependencies', 'internal', 'bug', 'breaking-change', 'enhancement', 'new feature'} |
* Update closure handlers documentaion (#736) * Rename closure callback/dropper types (#739) * Remove legacy z_zint_t from public API (#741) * Add `export "C"` for all header files (#740) * Add `export "C"` for proper closures callbacks export * Add `export "C"` for all headers * add serialize from substr functions (#742) * Rename z_loaned_hello_handler_t to z_closure_hello_callback_t (#743) * Mark z_reply_replier_id as unstable (#745) * Add documentation about logging (#744) * Update README.md (#746) * hardcode clang-format runner to Ubuntu24.04 (#748) * fix build granularity (#747) * fix: badly named constant (#750) * fix packages * fix: update debian packaging - include libzenohpico.so in debian package - change the package name to libzenohpico in line with libzenohc - set correct debian version for pre releases - fix wrong version used in Release mode * fix: debian dev package name * fix: package version for releases (#753) * fix: debian_version for official releases otherwise they would be undefined * fix: align cpack version with zenoh-c * chore: review comments move cpack version closer to where it's used * build shared lib for packages instead of static one (#757) * Fix z_task cleanup for platforms with pthread support (#759) * Fix read/lease task cleanup (#760) * Align ID string representation with zenoh (lowercase) (#761) * Remove default parameters from z_closure (#755) * build both libraries:shared and static when packaging; (#766) add static library to dev package; * Publish debian packages (#769) * fix: add workflow to release debian packages * fix: align with zenoh-c - update package names - set version string in the same way - set DEBARCH/RPMARCH - set CPACK_PACKAGE_FILE_NAME - add -j to zip archive to not include parent folder * fix: use CPACK_PACKAGE_NAME * fix: pass package name to all build targets * fix: Don't tag release branch during dry-run * Replace exit with return in zephyr examples (#774) * Add platform_common.c to zephyr CMakeLists.txt file * Rework Zenoh ID conversion * chore: Update org secrets (#782) As per eclipse-zenoh/.eclipsefdn#18, secrets were updated to follow eclipse foundation naming convention. * Implement liveliness support (#632) * fix: merge shenanigans * fix: integration test issues * fix: missing send reply final * fix: liveliness double free --------- Co-authored-by: Alexander Bushnev <Alexander@Bushnev.pro> Co-authored-by: DenisBiryukov91 <155981813+DenisBiryukov91@users.noreply.github.com> Co-authored-by: Denis Biryukov <denis.biryukov@zettascale.tech> Co-authored-by: Diogo Mendes Matsubara <diogo.matsubara@zettascale.tech> Co-authored-by: Luca Cominardi <luca.cominardi@gmail.com>
jean-roland
approved these changes
Nov 18, 2024
* Update closure handlers documentaion (#736) * Rename closure callback/dropper types (#739) * Remove legacy z_zint_t from public API (#741) * Add `export "C"` for all header files (#740) * Add `export "C"` for proper closures callbacks export * Add `export "C"` for all headers * add serialize from substr functions (#742) * Rename z_loaned_hello_handler_t to z_closure_hello_callback_t (#743) * Mark z_reply_replier_id as unstable (#745) * Add documentation about logging (#744) * Update README.md (#746) * hardcode clang-format runner to Ubuntu24.04 (#748) * fix build granularity (#747) * fix: badly named constant (#750) * fix packages * fix: update debian packaging - include libzenohpico.so in debian package - change the package name to libzenohpico in line with libzenohc - set correct debian version for pre releases - fix wrong version used in Release mode * fix: debian dev package name * fix: package version for releases (#753) * fix: debian_version for official releases otherwise they would be undefined * fix: align cpack version with zenoh-c * chore: review comments move cpack version closer to where it's used * build shared lib for packages instead of static one (#757) * Fix z_task cleanup for platforms with pthread support (#759) * Fix read/lease task cleanup (#760) * Align ID string representation with zenoh (lowercase) (#761) * Remove default parameters from z_closure (#755) * build both libraries:shared and static when packaging; (#766) add static library to dev package; * Publish debian packages (#769) * fix: add workflow to release debian packages * fix: align with zenoh-c - update package names - set version string in the same way - set DEBARCH/RPMARCH - set CPACK_PACKAGE_FILE_NAME - add -j to zip archive to not include parent folder * fix: use CPACK_PACKAGE_NAME * fix: pass package name to all build targets * fix: Don't tag release branch during dry-run * Replace exit with return in zephyr examples (#774) * Add platform_common.c to zephyr CMakeLists.txt file * Rework Zenoh ID conversion * chore: Update org secrets (#782) As per eclipse-zenoh/.eclipsefdn#18, secrets were updated to follow eclipse foundation naming convention. * Implement liveliness support (#632) * Mark liveliness API as unstable and disable by default * Fix typo (#787) * fix: correct 1.1 issues * feat: add zp_batch_flush * fix: switch to #if 0 to deactivate tcp function --------- Co-authored-by: Alexander Bushnev <Alexander@Bushnev.pro> Co-authored-by: DenisBiryukov91 <155981813+DenisBiryukov91@users.noreply.github.com> Co-authored-by: Denis Biryukov <denis.biryukov@zettascale.tech> Co-authored-by: Diogo Mendes Matsubara <diogo.matsubara@zettascale.tech> Co-authored-by: Luca Cominardi <luca.cominardi@gmail.com>
jean-roland
approved these changes
Nov 19, 2024
* Update closure handlers documentaion (#736) * Rename closure callback/dropper types (#739) * Remove legacy z_zint_t from public API (#741) * Add `export "C"` for all header files (#740) * Add `export "C"` for proper closures callbacks export * Add `export "C"` for all headers * add serialize from substr functions (#742) * Rename z_loaned_hello_handler_t to z_closure_hello_callback_t (#743) * Mark z_reply_replier_id as unstable (#745) * Add documentation about logging (#744) * Update README.md (#746) * hardcode clang-format runner to Ubuntu24.04 (#748) * fix build granularity (#747) * fix: badly named constant (#750) * fix packages * fix: update debian packaging - include libzenohpico.so in debian package - change the package name to libzenohpico in line with libzenohc - set correct debian version for pre releases - fix wrong version used in Release mode * fix: debian dev package name * fix: package version for releases (#753) * fix: debian_version for official releases otherwise they would be undefined * fix: align cpack version with zenoh-c * chore: review comments move cpack version closer to where it's used * build shared lib for packages instead of static one (#757) * Fix z_task cleanup for platforms with pthread support (#759) * Fix read/lease task cleanup (#760) * Align ID string representation with zenoh (lowercase) (#761) * Remove default parameters from z_closure (#755) * build both libraries:shared and static when packaging; (#766) add static library to dev package; * Publish debian packages (#769) * fix: add workflow to release debian packages * fix: align with zenoh-c - update package names - set version string in the same way - set DEBARCH/RPMARCH - set CPACK_PACKAGE_FILE_NAME - add -j to zip archive to not include parent folder * fix: use CPACK_PACKAGE_NAME * fix: pass package name to all build targets * fix: Don't tag release branch during dry-run * Replace exit with return in zephyr examples (#774) * Add platform_common.c to zephyr CMakeLists.txt file * Rework Zenoh ID conversion * chore: Update org secrets (#782) As per eclipse-zenoh/.eclipsefdn#18, secrets were updated to follow eclipse foundation naming convention. * Implement liveliness support (#632) * Mark liveliness API as unstable and disable by default * Fix typo (#787) * fix: prevent code duplication during merge --------- Co-authored-by: Alexander Bushnev <Alexander@Bushnev.pro> Co-authored-by: DenisBiryukov91 <155981813+DenisBiryukov91@users.noreply.github.com> Co-authored-by: Denis Biryukov <denis.biryukov@zettascale.tech> Co-authored-by: Diogo Mendes Matsubara <diogo.matsubara@zettascale.tech> Co-authored-by: Luca Cominardi <luca.cominardi@gmail.com>
Fix merge main
Dev/1.1.0 merge main
Fix merge issue
sashacmc
approved these changes
Nov 20, 2024
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
merge dev/1.1.0 branch into main