Skip to content

Conversation

@jacque1ine
Copy link
Collaborator

No description provided.

@KeranYang
Copy link
Contributor

Thank you @jacque1ine for writing detailed user guide. Just a few high level suggestions - 1. we now have two user guides, one is the README.md and this one. Let's keep using README.md as the guide and change this one to something like running pulsar on StreamNative. 2. we already have two folders, one for source and one for sink. User should always go to one of them for creating a pipeline. I see that in both files, we have a section to Have a Pulsar cluster running, can we have change this PR to setup StreamNative on the Have a Pulsar cluster running section, and then update the following steps to add auth? Basically merge user guide to source and sink guides.

jacque1ine and others added 4 commits April 8, 2025 14:16
Signed-off-by: jacque1ine <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Keran Yang <[email protected]>
Signed-off-by: jacque1ine <[email protected]>
Signed-off-by: jacque1ine <[email protected]>
Signed-off-by: jacque1ine <[email protected]>
@jacque1ine jacque1ine requested a review from Copilot April 8, 2025 18:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

docs/source/byte-array/manifests/api-key/byte-arr-consumer-pipeline.yaml:4

  • The pipeline is defined in a consumer context but is named 'raw-producer-pipeline'. Consider renaming it to 'raw-consumer-pipeline' (or similar) to accurately reflect its purpose.
  name: raw-producer-pipeline

@KeranYang KeranYang merged commit 4cbaa28 into numaproj-contrib:main Apr 9, 2025
2 checks passed
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