Merging dev/1.0.0 into main #176
Closed
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
* issue(protocol changes): work in progress - successful build * issue(protocol changes): wip - exposing encoding schema to Kotlin * issue(protocol changes): wip - fixing Get operation * issue(protocol changes): wip - QoS feature refactor * issue(protocol changes): wip - cargo update + query reply refactor * issue(protocol changes): wip - fix query bug Encoding id was wrongly passed to the JNI layer. * issue(protocol changes): receiving encoding schema on subscriber callback * issue(protocol changes): receiving encoding schema on get operations * issue(protocol changes): receiving encoding schema on queries with value * issue(protocol changes): enabling error replies on queries + receiving encoding id and schema on get queries (successful and error replies). * issue(protocol changes): enabling DELETE replies on queries * issue(protocol changes): fix + refactoring session delete * issue(protocol changes): removing Attachment.kt, replacing the Attachment class with raw ByteArray. * issue(protocol changes): cargo update * issue(protocol changes): fix cargo clippy * issue(protocol changes): removing unwraps * issue(protocol changes): fix format * Cargo update
… express config to publisher builder (eclipse-zenoh#110) * issue(Publisher QoS): adding express config for the Publisher. * issue(Publisher QoS): removing publisher setPriority and setCongestionControl during runtime. Done after changes on Zenoh from eclipse-zenoh/zenoh#1124 * issue(Publisher QoS): refactoring publisher.rs on zenoh-jni
…-bot/sync-toolchain Sync Rust toolchain
…own the args parsing logic of all the examples (eclipse-zenoh#111)
* refactor(jni): functions reordering * refactor(jni): modif throw_exception macro * refactor(jni): modifying visibility of key expr functions * refactor(jni): logger.rs * refactor(jni): queryable refactor * refactor(jni): key expr declaration * refactor(jni): moving reply logic below getViaJNI function * refactor(jni): fix session ptr being liberated in case of publisher declaration error * refactor(jni): fixes after cargo update + creating ZenohID class * refactor(jni): using macros for errors * refactor(jni): using AutoLocal instead of manually deleting local references * refactor(jni): key expr inner code refactor * refactor(jni): tidying up jni comments * refactor(jni): cargo fmt + clippy
Cargo update + bumping version
…ipse-zenoh#127) * Enabling snapshot publications for dev/1.0.0 * refactor(gradle): using 'android' property to enable/disable Android configuration Examples: - `gradle build -Pandroid=true` enables build with android - `gradle build` won't consider any android configuration - `gradle publishAndroidReleaseToMavenLocal -Pandroid=true` publishes to maven local, without `-Pandroid=true` it won't work. * refactor(gradle): updating publication workflows * refactor(gradle): updating readme instructions --------- Co-authored-by: zettascale-bot <[email protected]>
…ishing jvm to maven local (eclipse-zenoh#128)
…-bot/sync-toolchain Sync Rust toolchain
…-bot/sync-toolchain Sync Rust toolchain
* chore: Update setup-java to v4 * chore: update gradle action to newer version * chore: update gradle/wrapper-validation-action@v3
…ipse-zenoh#135) * fix: Update buildDir usage for gradle 8 * Update gradle wrapper action to newer version * calling gradle action via arguments is deprecated * fix: remove unnecessary call to buildDir() * fix: add -Pandroid to gradle dokkaHtml call Otherwise the docs won't build
…sion. (eclipse-zenoh#96) * feat(background declarations): Allowing session declarations to stay alive for the lifespan of a session. Zenoh declarations ran for as long as the Kotlin variable representing them was kept alive. This meant that whenever the user lost track of the variable, it got garbage collected and undeclared in the process. This behavior seems to be counterintuitive for programmers used to garbage collected languages (see eclipse-zenoh#43). Therefore in this PR we provide the following change: we keep track of session declarations in a list inside the session, allowing users to keep running them despite losing their references. When the session is finalized, the associated declarations are undeclared. In case the user needs to close a declaration earlier, they need to keep the variable in order to undeclare it. * feat(background declarations): closing all session declarations immediately after doing `session.close()`
…-zenoh#150) Co-authored-by: eclipse-zenoh-bot <[email protected]>
Bumping version to 1.0.0.alpha-dev
…ev/1.0.0.alpha Revert "Bumping version to 1.0.0.alpha-dev"
* build: Sync with eclipse-zenoh/zenoh@9d09742 from 2024-06-11 (eclipse-zenoh#109) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@ed6c636 from 2024-06-12 (eclipse-zenoh#112) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@8160b01 from 2024-06-13 (eclipse-zenoh#115) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Enable releasing from any branch (eclipse-zenoh#114) * build: Sync with eclipse-zenoh/zenoh@7adad94 from 2024-06-14 (eclipse-zenoh#117) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@93f93d2 from 2024-06-17 (eclipse-zenoh#118) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@2500e5a from 2024-06-20 (eclipse-zenoh#119) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@869ace6 from 2024-07-02 (eclipse-zenoh#120) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@b93ca84 from 2024-07-03 (eclipse-zenoh#121) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Enabling snapshot publications for dev/1.0.0 on the CI * fix: wrong syntax in ci workflow (eclipse-zenoh#124) * build: Sync with eclipse-zenoh/zenoh@b3e42ce from 2024-07-08 (eclipse-zenoh#126) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Update main release workflow (eclipse-zenoh#133) * Using 'android' property to enable/disable Android configuration (eclipse-zenoh#127) * Enabling snapshot publications for dev/1.0.0 * refactor(gradle): using 'android' property to enable/disable Android configuration Examples: - `gradle build -Pandroid=true` enables build with android - `gradle build` won't consider any android configuration - `gradle publishAndroidReleaseToMavenLocal -Pandroid=true` publishes to maven local, without `-Pandroid=true` it won't work. * refactor(gradle): updating publication workflows * refactor(gradle): updating readme instructions --------- Co-authored-by: zettascale-bot <[email protected]> * fix(local publication): triggering compilation of zenoh jni when publishing jvm to maven local (eclipse-zenoh#128) * Update actions (eclipse-zenoh#130) * chore: Update setup-java to v4 * chore: update gradle action to newer version * chore: update gradle/wrapper-validation-action@v3 --------- Co-authored-by: Darius Maitia <[email protected]> Co-authored-by: zettascale-bot <[email protected]> * fix: Update gradle actions to newer versions (eclipse-zenoh#134) * fix: Update buildDir usage for gradle 8 * Update gradle wrapper action to newer version * calling gradle action via arguments is deprecated * fix: remove unnecessary call to buildDir() * fix: add -Pandroid to gradle dokkaHtml call Otherwise the docs won't build * Fix documentation publication error on CI (eclipse-zenoh#136) * build: Sync with eclipse-zenoh/zenoh@0a969cb from 2024-07-25 (eclipse-zenoh#141) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@e587aa9 from 2024-07-26 (eclipse-zenoh#144) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@2d88c7b from 2024-07-29 (eclipse-zenoh#147) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Fix JVM crash when finalizing a session that wasn't properly initialized (eclipse-zenoh#139) (eclipse-zenoh#140) * Fix snapshot publications on the CI (eclipse-zenoh#149) * Fix ZPubThr example imports (eclipse-zenoh#142) --------- Co-authored-by: eclipse-zenoh-bot <[email protected]> Co-authored-by: eclipse-zenoh-bot <[email protected]> Co-authored-by: Mahmoud Mazouz <[email protected]> Co-authored-by: kydos <[email protected]> Co-authored-by: Diogo Matsubara <[email protected]> Co-authored-by: zettascale-bot <[email protected]> Co-authored-by: Jonathan Hepp <[email protected]>
…-zenoh#155) Co-authored-by: eclipse-zenoh-bot <[email protected]>
…-zenoh#156) Co-authored-by: eclipse-zenoh-bot <[email protected]>
…-zenoh#157) Co-authored-by: eclipse-zenoh-bot <[email protected]>
* Change default consolidation mode to AUTO * Implement default function for ConsolidationMode
…-zenoh#166) Co-authored-by: eclipse-zenoh-bot <[email protected]>
When processing the internal string representation of a kotlin KeyExpr instance through JNI, a rust KeyExpr instance was generated through the "try_from" function. This is unnecessary since the string representation was already validated during the construction phase of the KeyExpr object on Kotlin. Therefore, this commit introduces an optimization in which the rust key expression is created with "from_string_unchecked", since we already guarantee that the string representation provided is valid and has already been checked.
* feat(api alignment): replacing res() with wait() * feat(api alignment): replacing `withValue(...)` with `.payload(...)` on get queries * feat(api alignment): making 'value' attribute internal for Query, offering 'payload' and 'encoding' accessors instead. * feat(api alignment): using ZBytes for Value instead of ByteArray * feat(api alignment): deserialization for String and Int * feat(api alignment): deserialization for Byte, Short & Long * feat(api alignment): deserialization for Float & Double * feat(api alignment): deserialization * feat(api alignment): serialization - wip * feat(api alignment): serialization - wip 2 * feat(api alignment): custom deserializers - wip * feat(api_alignment): zbytes jni refactor * feat(api alignment): fixing attachment api * feat(api alignment): ZBytes example + custom serialization improvements * feat(api_alignment): serializable maps * feat(api alignment): serialization refactor * feat(api alignment): deserialization refactor * rebase fix * feat(api alignment): deserialization refactor * feat(api alignment): splitting Serializable interface into Serializable and Deserializable. * feat(api alignment): marking DeserializationUtils with @PublishedApi tag * feat(api alignment): fix clippy errors * feat(api alignment): kdoc comment for ZBytes + removing Serializable for ZBytes * feat(api alignment): fix imports * feat(api alignment): Removing IntoZBytes interface (in favor of Serializable) + moving Into ZBytes extension functions under ZBytes.kt * feat(api alignment): deserialization refactor * feat(api alignment): moving Serializable and Deserializable under their own file. * feat(api alignment): refactoring ZBytesTest.kt * feat(api alignment): refactoring ZBytes and ZBytesTest * feat(default args): subscriber - wip * feat(default args): publisher * feat(default args): subscriber - updating documentation * feat(default args): key expressions - removing resolvables * feat(default args): queryable * feat(default args): get query - WIP * feat(default args): put operation - WIP * feat(default args): delete operation - WIP * feat(default args): fixing examples * feat(default args): changing receiver attribute from Queryable, Get and Subscriber to be non optional. * fix(selector): setting selector parameters as a nullable argument instead of putting an empty string. * fix(selector): setting selector parameters as a nullable argument instead of putting an empty string. * feat(default arguments): replacing reply builders with default args * feat(default arguments): removing QoS builder * feat(default arguments): making ChannelHandler internal * feat(default arguments): removing the Get class (unused after changes) * feat(default arguments): making Encoding and QoS data classes * feat(default arguments): making Sample a data class * feat(default arguments): updating KDoc for session.get . * feat(default arguments): updating KDoc for session.declareQueryable * feat(default arguments): updating KDoc for session.put and session.delete * feat(default arguments): updating KDoc for Reply, converting Reply.Success, Reply.Error & Reply.Delete into data classes. * Merge 'main' into 'dev/1.0.0' (eclipse-zenoh#153) * build: Sync with eclipse-zenoh/zenoh@9d09742 from 2024-06-11 (eclipse-zenoh#109) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@ed6c636 from 2024-06-12 (eclipse-zenoh#112) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@8160b01 from 2024-06-13 (eclipse-zenoh#115) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Enable releasing from any branch (eclipse-zenoh#114) * build: Sync with eclipse-zenoh/zenoh@7adad94 from 2024-06-14 (eclipse-zenoh#117) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@93f93d2 from 2024-06-17 (eclipse-zenoh#118) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@2500e5a from 2024-06-20 (eclipse-zenoh#119) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@869ace6 from 2024-07-02 (eclipse-zenoh#120) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@b93ca84 from 2024-07-03 (eclipse-zenoh#121) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Enabling snapshot publications for dev/1.0.0 on the CI * fix: wrong syntax in ci workflow (eclipse-zenoh#124) * build: Sync with eclipse-zenoh/zenoh@b3e42ce from 2024-07-08 (eclipse-zenoh#126) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Update main release workflow (eclipse-zenoh#133) * Using 'android' property to enable/disable Android configuration (eclipse-zenoh#127) * Enabling snapshot publications for dev/1.0.0 * refactor(gradle): using 'android' property to enable/disable Android configuration Examples: - `gradle build -Pandroid=true` enables build with android - `gradle build` won't consider any android configuration - `gradle publishAndroidReleaseToMavenLocal -Pandroid=true` publishes to maven local, without `-Pandroid=true` it won't work. * refactor(gradle): updating publication workflows * refactor(gradle): updating readme instructions --------- Co-authored-by: zettascale-bot <[email protected]> * fix(local publication): triggering compilation of zenoh jni when publishing jvm to maven local (eclipse-zenoh#128) * Update actions (eclipse-zenoh#130) * chore: Update setup-java to v4 * chore: update gradle action to newer version * chore: update gradle/wrapper-validation-action@v3 --------- Co-authored-by: Darius Maitia <[email protected]> Co-authored-by: zettascale-bot <[email protected]> * fix: Update gradle actions to newer versions (eclipse-zenoh#134) * fix: Update buildDir usage for gradle 8 * Update gradle wrapper action to newer version * calling gradle action via arguments is deprecated * fix: remove unnecessary call to buildDir() * fix: add -Pandroid to gradle dokkaHtml call Otherwise the docs won't build * Fix documentation publication error on CI (eclipse-zenoh#136) * build: Sync with eclipse-zenoh/zenoh@0a969cb from 2024-07-25 (eclipse-zenoh#141) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@e587aa9 from 2024-07-26 (eclipse-zenoh#144) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@2d88c7b from 2024-07-29 (eclipse-zenoh#147) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Fix JVM crash when finalizing a session that wasn't properly initialized (eclipse-zenoh#139) (eclipse-zenoh#140) * Fix snapshot publications on the CI (eclipse-zenoh#149) * Fix ZPubThr example imports (eclipse-zenoh#142) --------- Co-authored-by: eclipse-zenoh-bot <[email protected]> Co-authored-by: eclipse-zenoh-bot <[email protected]> Co-authored-by: Mahmoud Mazouz <[email protected]> Co-authored-by: kydos <[email protected]> Co-authored-by: Diogo Matsubara <[email protected]> Co-authored-by: zettascale-bot <[email protected]> Co-authored-by: Jonathan Hepp <[email protected]> * build: Sync with eclipse-zenoh/zenoh@b1e4dba from 2024-08-05 (eclipse-zenoh#155) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@b7d42ef from 2024-08-06 (eclipse-zenoh#156) Co-authored-by: eclipse-zenoh-bot <[email protected]> * build: Sync with eclipse-zenoh/zenoh@2d5ab7c from 2024-08-06 (eclipse-zenoh#157) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Add Query `Consolidation::Auto` to JNI and Kotlin API (eclipse-zenoh#158) * Add Consolidation::Auto to JNI and Kotlin API * Use default consolidation in ZGet example * Change default consolidation mode to AUTO (eclipse-zenoh#159) * Change default consolidation mode to AUTO * Implement default function for ConsolidationMode * build: Sync with eclipse-zenoh/zenoh@0e2f78a from 2024-08-14 (eclipse-zenoh#166) Co-authored-by: eclipse-zenoh-bot <[email protected]> * Optimizing KeyExpr processing (eclipse-zenoh#161) When processing the internal string representation of a kotlin KeyExpr instance through JNI, a rust KeyExpr instance was generated through the "try_from" function. This is unnecessary since the string representation was already validated during the construction phase of the KeyExpr object on Kotlin. Therefore, this commit introduces an optimization in which the rust key expression is created with "from_string_unchecked", since we already guarantee that the string representation provided is valid and has already been checked. * feat(api alignment): replacing `withValue(...)` with `.payload(...)` on get queries * feat(api-alignment): removing Resolvable.kt --------- Co-authored-by: eclipse-zenoh-bot <[email protected]> Co-authored-by: eclipse-zenoh-bot <[email protected]> Co-authored-by: Mahmoud Mazouz <[email protected]> Co-authored-by: kydos <[email protected]> Co-authored-by: Diogo Matsubara <[email protected]> Co-authored-by: zettascale-bot <[email protected]> Co-authored-by: Jonathan Hepp <[email protected]> Co-authored-by: oteffahi <[email protected]>
This reverts commit c6aff70.
* feat(api alignment): replacing res() with wait() * feat(api alignment): replacing `withValue(...)` with `.payload(...)` on get queries * feat(api alignment): making 'value' attribute internal for Query, offering 'payload' and 'encoding' accessors instead. * feat(api alignment): using ZBytes for Value instead of ByteArray * feat(api alignment): deserialization for String and Int * feat(api alignment): deserialization for Byte, Short & Long * feat(api alignment): deserialization for Float & Double * feat(api alignment): deserialization * feat(api alignment): serialization - wip * feat(api alignment): serialization - wip 2 * feat(api alignment): custom deserializers - wip * feat(api_alignment): zbytes jni refactor * feat(api alignment): fixing attachment api * feat(api alignment): ZBytes example + custom serialization improvements * feat(api_alignment): serializable maps * feat(api alignment): serialization refactor * feat(api alignment): deserialization refactor * rebase fix * feat(api alignment): deserialization refactor * feat(api alignment): splitting Serializable interface into Serializable and Deserializable. * feat(api alignment): marking DeserializationUtils with @PublishedApi tag * feat(api alignment): fix clippy errors * feat(api alignment): kdoc comment for ZBytes + removing Serializable for ZBytes * feat(api alignment): fix imports * feat(api alignment): Removing IntoZBytes interface (in favor of Serializable) + moving Into ZBytes extension functions under ZBytes.kt * feat(api alignment): deserialization refactor * feat(api alignment): moving Serializable and Deserializable under their own file. * feat(api alignment): refactoring ZBytesTest.kt * feat(api alignment): refactoring ZBytes and ZBytesTest * feat(default args): subscriber - wip * feat(default args): publisher * feat(default args): subscriber - updating documentation * feat(default args): key expressions - removing resolvables * feat(default args): queryable * feat(default args): get query - WIP * feat(default args): put operation - WIP * feat(default args): delete operation - WIP * feat(default args): fixing examples * feat(default args): changing receiver attribute from Queryable, Get and Subscriber to be non optional. * fix(selector): setting selector parameters as a nullable argument instead of putting an empty string. * feat(default arguments): replacing reply builders with default args * feat(default arguments): removing QoS builder * feat(default arguments): making ChannelHandler internal * feat(default arguments): removing the Get class (unused after changes) * feat(default arguments): making Encoding and QoS data classes * feat(default arguments): making Sample a data class * feat(default arguments): updating KDoc for session.get . * feat(default arguments): updating KDoc for session.declareQueryable * feat(default arguments): updating KDoc for session.put and session.delete * feat(default arguments): updating KDoc for Reply, converting Reply.Success, Reply.Error & Reply.Delete into data classes. * feat(api alignment): replacing `withValue(...)` with `.payload(...)` on get queries * feat(api-alignment): removing Resolvable.kt * Removing IntoZBytes.kt * feat(api-alignment): KDoc fixes * fix: invalid query pointer freed
* chore: Bump version to * chore: Bump /zenoh.*/ dependencies to `1.0.0-dev` --------- Co-authored-by: eclipse-zenoh-bot <[email protected]>
…-zenoh#175) Co-authored-by: eclipse-zenoh-bot <[email protected]>
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.
No description provided.