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

Generate unique positions #17

Merged
merged 3 commits into from
Jul 12, 2022
Merged

Generate unique positions #17

merged 3 commits into from
Jul 12, 2022

Conversation

hariso
Copy link
Contributor

@hariso hariso commented Jul 11, 2022

Description

Makes positions unique. Works around ConduitIO/conduit#517.

The ability to run multiple generators will be useful for the perf. benchmarks.

Quick checks:

  • I have followed the Code Guidelines.
  • There is no other pull request for the same update/change.
  • I have written unit tests.
  • I have made sure that the PR is of reasonable size and can be easily reviewed.

@hariso hariso marked this pull request as ready for review July 11, 2022 15:52
@hariso hariso requested a review from a team as a code owner July 11, 2022 15:52
go.mod Outdated
@@ -4,6 +4,7 @@ go 1.17

require (
github.com/conduitio/conduit-connector-sdk v0.2.0
github.com/google/uuid v1.1.2
Copy link
Member

Choose a reason for hiding this comment

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

We should use the latest version though, v1.3.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't check tbh, I thought it would automatically pull the latest version :S

Copy link
Member

Choose a reason for hiding this comment

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

I think go get fetches the latest version, unless you already have one locally, then it takes that one. You can use go get my-dependency@latest to always fetch the latest.

@hariso hariso merged commit 42aa37f into main Jul 12, 2022
@hariso hariso deleted the haris/unique-positions branch July 12, 2022 11:53
@hariso hariso restored the haris/unique-positions branch July 12, 2022 12:31
@lovromazgon lovromazgon deleted the haris/unique-positions branch June 20, 2024 14: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