Skip to content

Broadcast Protocol#115

Merged
zonotope merged 31 commits intomainfrom
feature/broadcast
Feb 20, 2025
Merged

Broadcast Protocol#115
zonotope merged 31 commits intomainfrom
feature/broadcast

Conversation

@zonotope
Copy link
Contributor

This patch has changes that make it easier to use Server for the Nexus use case.

  • adds a Broadcaster protocol to better abstract reporting transactions to interested clients
  • adds iris for broadcaster properties and types to the configuration vocabulary
  • refactors the command line options parsing so it can be used by dependent systems
  • simplifies the consensus event message format and makes it more consistent by using a single map with a type key for every message instead of inconsistently using two element vectors with a type as the first entry an extra data map as the second

It also includes some cleanup, reorganization, and improvements:

  • removes the redundancy from the server entry point namespace by changing it to fluree.server from fluree.server.main.
  • moves broadcast related namespaces out of the consensus hierarchy
  • changes the Makefile to be self-documenting, and adds a help task to list and describe the available tasks, like this:
$ make help
benchmark       Benchmark performance
clean           Remove build artifacts
cljfmt-check    Check Clojure formatting with cljfmt
clj-kondo-lint  Lint Clojure code with clj-kondo
docker-build    Build the server docker container
docker-push     Build and publish the server docker container
docker-run      Run in docker
help            Describe available tasks
pending-tests   Run pending tests
prepare         Prepare clojure dependencies
test            Run tests
uberjar         Build an executable server uberjar

@zonotope zonotope requested a review from a team February 20, 2025 17:17
@zonotope zonotope self-assigned this Feb 20, 2025
Copy link
Contributor

@dpetran dpetran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗼

@zonotope zonotope merged commit 4bb0c1c into main Feb 20, 2025
5 checks passed
@zonotope zonotope deleted the feature/broadcast branch February 20, 2025 21:47
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.

2 participants