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

Pulsar #121

Merged
merged 102 commits into from
Apr 4, 2023
Merged

Pulsar #121

merged 102 commits into from
Apr 4, 2023

Conversation

jbellis
Copy link
Collaborator

@jbellis jbellis commented Mar 16, 2023

draft for review

jbellis and others added 18 commits March 17, 2023 10:50
@jbellis
Merge branch 'main' into pulsar
c4bb219
@epinzur @jbellis
proposed proto changes to support pulsar sources
28f3d3a
@jbellis
add admin_service_url, auth_plugin; replace auth_token with auth_params
bac5b36
@jbellis
add to destination as well as source
c7b00b3
@jbellis
add new proto files to build.rs
b1771e2
@jordanrfrazier
explicitly allow MPL2.0
ab4247c
@jordanrfrazier
Merge pull request #127 from kaskada-ai/pulsar-whitelist-oauth2
1ab4172 
explicitly allow MPL2.0
@jordanrfrazier
Add jwt token auth to pulsar output
655ac7b
@jbellis
plumb pulsar connection proto through prepare, and test that it works…
1ec18f8 
… with Astra
@jbellis
Merge branch 'pulsar' of github.com:kaskada-ai/kaskada into pulsar
8b04621
@jbellis
fix most tests
990d17e
@jordanrfrazier
remove comment
101d2d7
@jordanrfrazier
Merge pull request #129 from kaskada-ai/pulsar-add-auth-to-output
1dd4072 
Add jwt token auth to pulsar output
@jordanrfrazier
Add cert chain field to pulsar
affc3f1
@jordanrfrazier
update number
b4277be
@rltvty
Merge pull request #134 from kaskada-ai/pulsar-add-tls-configs
29fb802 
Add cert chain field to pulsar
@jbellis
merge from main
64f15ca
@jbellis
add missing cert chain field
66004b3
jbellis and others added 15 commits March 30, 2023 12:41
@jbellis
make proto lint happy
3fc7188
@jbellis
clippy
1de3e30
@epinzur
marked the pulsar auth_params field as sensitive
ce5be18
@rltvty
Merge pull request #192 from kaskada-ai/puslar/sensitive
a5b250e 
marked the pulsar auth_params field as sensitive
@jbellis
implement Stream for PulsarReader using async_stream::stream!
4790f1c
@bjchambers
example: use async stream directly
eb4a675
@bjchambers
example with try unfold
5f9a96b
@jbellis
rewrite prepare_file to use filter_map and unzip instead of manually …
8d58743 
…pushing into arc-mutex-vec
@jbellis
merge async_stream_reader
d6b813b
@jordanrfrazier
remove cert chain and redact auth from log
1ea69cf
@rltvty
Merge pull request #197 from kaskada-ai/remove-cert-and-auth
d17b016 
remove cert chain and redact auth from log
@jordanrfrazier
remove allowed security advisories
428ae2f
@jordanrfrazier
Remove lingering uses of cert chain in python
6f962a5
@jordanrfrazier
Attempt to fix prepare span tracing
6a8c4d1
@jordanrfrazier
remove logging
10f30a9
@jordanrfrazier jordanrfrazier added this pull request to the merge queue Apr 4, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Apr 4, 2023
@jordanrfrazier jordanrfrazier added this pull request to the merge queue Apr 4, 2023
Merged via the queue into main with commit 79b2d94 Apr 4, 2023
@jordanrfrazier jordanrfrazier deleted the pulsar branch April 4, 2023 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jordanrfrazier jordanrfrazier jordanrfrazier approved these changes

@kevinjnguyen kevinjnguyen kevinjnguyen left review comments

@epinzur epinzur epinzur left review comments

@bjchambers bjchambers bjchambers left review comments

Assignees
No one assigned
Labels
proto python sparrow wren
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jbellis @jordanrfrazier @bjchambers @epinzur @kevinjnguyen @rltvty