Skip to content

Expose opaque-tier fields needed by generated JNI#1

Merged
milyin merged 1 commit into
mainfrom
prebindgen-split-api
Jul 15, 2026
Merged

Expose opaque-tier fields needed by generated JNI#1
milyin merged 1 commit into
mainfrom
prebindgen-split-api

Conversation

@milyin

@milyin milyin commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • preserve the original base / expanded two-tier source structure
  • add only the missing opaque z_* accessors needed for one-crossing JNI callback decomposition
  • add numeric-id encoding construction and with_schema on the opaque encoding tier
  • expose borrowed key-expression text and Rust error messages to language adapters
  • fix the existing unstable-feature throughput example call

Ownership

No existing session or key-expression signature changes. In particular, z_open(ZConfig) still consumes its configuration exactly like zenoh::open, and declaration functions still consume their ZKeyExpr inputs.

This is the flat source for ZettaScaleLabs/zenoh-flat-jni#2 and eclipse-zenoh/zenoh-java#481.

Validation

  • cargo check --all-features
  • cargo test --all-targets --all-features
  • clippy with the two existing main-branch lints allowed: module inception from the original layout and manual is-multiple-of in the existing throughput example
  • full downstream 101-test JVM suite against prebindgen main

@milyin

milyin commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator Author

Coordinated follow-ups: ZettaScaleLabs/zenoh-flat-jni#2 and eclipse-zenoh/zenoh-java#481.

@milyin
milyin force-pushed the prebindgen-split-api branch from 49033f7 to 9d404a7 Compare July 15, 2026 15:55
@milyin milyin changed the title Borrow reusable config and key expression inputs Expose opaque-tier fields needed by generated JNI Jul 15, 2026
@milyin
milyin merged commit 06c13d0 into main Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant