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.

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

Remove global ,format Topics , consumer options and base test suite #2

Merged
merged 10 commits into from
Aug 27, 2023

Conversation

avrahams
Copy link
Contributor

@avrahams avrahams commented Aug 24, 2023

Remove global client and config
Add Client, Consumer and Producers interfaces that wraps pulsar types
Add Client methods to create consumers and producers
Handle topics format in Producer and consumer constructors
Improve consumer options
Create base test suite

@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

@Bezbran
Copy link
Contributor

Bezbran commented Aug 24, 2023

/describe

pulsar/test/utils.go Show resolved Hide resolved
pulsar/connector/consumer.go Show resolved Hide resolved
pulsar/connector/consumer.go Outdated Show resolved Hide resolved
pulsar/connector/producer.go Outdated Show resolved Hide resolved
Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

@avrahams avrahams requested a review from Bezbran August 24, 2023 13:30
pulsar/connector/consumer.go Outdated Show resolved Hide resolved
pulsar/connector/producer.go Outdated Show resolved Hide resolved
Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
@avrahams avrahams requested a review from Bezbran August 24, 2023 14:34
@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

Bezbran
Bezbran previously approved these changes Aug 27, 2023
Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
@avrahams avrahams changed the title Topics format, consumer options and base test suite Remove global ,format Topics , consumer options and base test suite Aug 27, 2023
@avrahams avrahams requested a review from Bezbran August 27, 2023 08:24
Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
Signed-off-by: Avraham Shalev <8184528+avrahams@users.noreply.github.com>
@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

1 similar comment
@github-actions
Copy link

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

@avrahams avrahams merged commit 4f24edf into main Aug 27, 2023
8 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.

None yet

2 participants