Skip to content

Commit

Permalink
refactor(smart module): rationalize smart-module dependencies (#2579)
Browse files Browse the repository at this point in the history
To simplify SmartModule-related infrastructure, the following changes are made:
* Remove `fluvio-dataplane-protocol` crate.   Parts of the crates were moved to `fluvio-protocol` with a feature flag.
* API-related codes are moved to `fluvio-spu-schema` crate.
* SmartModule specific codes are moved to `fluvio-smaretmodule` crate.
* Engine-related codes are moved to `fluvio-smartengine.`

Both `fluvio-smartmodule` and `fluvio-smartengine` have feature flags with no default features.   No default feature-based code is intended to be shared with other crates.   This makes it easier to co-locate code based on the use case.

These changes reduce internal dependencies, which reduce coupling between crates and make it easier to maintain codes.
  • Loading branch information
sehz committed Aug 24, 2022
1 parent 167694a commit 3d5f0ec
Show file tree
Hide file tree
Showing 221 changed files with 2,150 additions and 2,423 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/ci.yml
Expand Up @@ -117,6 +117,7 @@ jobs:
if: github.event_name == 'push' && github.ref == 'refs/heads/staging'
runs-on: ${{ matrix.os }}
strategy:
# fail-fast: false
matrix:
rust-target:
- aarch64-apple-darwin
Expand Down Expand Up @@ -350,7 +351,6 @@ jobs:
- fluvio-protocol
- fluvio-spu-schema
- fluvio-sc-schema
- fluvio-dataplane-protocol
- fluvio-types
env:
RUST_BACKTRACE: full
Expand Down Expand Up @@ -384,7 +384,6 @@ jobs:
- fluvio-sc-schema
- fluvio-protocol
- fluvio-spu-schema
- fluvio-dataplane-protocol
- fluvio-types
features:
- no-default-features
Expand Down

0 comments on commit 3d5f0ec

Please sign in to comment.