Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Refine Managed Subscribe Sample #44

Merged
merged 15 commits into from
Sep 26, 2023
Merged

Refine Managed Subscribe Sample #44

merged 15 commits into from
Sep 26, 2023

Conversation

devkelley
Copy link
Contributor

The is PR does the following:

  • Adds a README explaining the managed_subscribe sample with sequence diagrams.
  • Updates the consumer to allow for the frequency constraint to be set by the command line call.
  • Allows for multiple consumers to be run.
  • Consumer now shuts down more efficiently.
  • Adds comments to managed subscribe proto file.

@devkelley
Renamed extension to module, Restructured so each module has own Carg…
f04a661 
…o.toml, Added GrpcServer
@devkelley
Updated docs
432348f
@devkelley
Updated comments
fc9a23d
@devkelley
change managed_subscribe_ext to managed_subscribe_module and removed …
dbdfa06 
…extension from comments.
@devkelley
Cleaned up managed sub sample, added readme
282a71c
@devkelley
updated readme
c90f63b
@devkelley
Update readme and add proto comments
c71a2a5
@devkelley
Merge branch 'eclipse-ibeji:main' into devkelley/standardize_extensio…
dd283ea 
…n_model
@devkelley
Merge remote-tracking branch 'origin/devkelley/standardize_extension_…
9cacd93 
…model' into devkelley/refine_managed_subscribe_sample
@devkelley
add file end space
0138ed4
@devkelley
split out sequence diagrams into interceptor and module
f2c0444
@devkelley
fix proto error and added readme to spell_check
3b071bf
@devkelley
fix linting error
679d42c
@devkelley
remove trailing whitespace
3e9f599
@jorchiu jorchiu merged commit 60116b3 into eclipse-ibeji:main Sep 26, 2023
mobicatk pushed a commit to Mobica/EclipseSDV_ibeji that referenced this pull request Apr 24, 2024
@devkelley
Refine Managed Subscribe Sample (eclipse-ibeji#44)
0a09101
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jorchiu jorchiu jorchiu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@devkelley @jorchiu