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

Producer implementation #13

Merged
merged 18 commits into from
Jan 15, 2025
Merged

Producer implementation #13

merged 18 commits into from
Jan 15, 2025

Conversation

Nayrode
Copy link
Collaborator

@Nayrode Nayrode commented Jan 14, 2025

changing structure + implementing producer

@Nayrode Nayrode linked an issue Jan 14, 2025 that may be closed by this pull request
Producer Implementation #6
Closed
@Nayrode Nayrode force-pushed the 6-producer-implementation branch 3 times, most recently from 945c82d to a9df315 Compare January 14, 2025 12:28
Razano26
Razano26 reviewed Jan 14, 2025
View reviewed changes
.gitignore Outdated Show resolved Hide resolved
@dorian-grst dorian-grst changed the title 6 producer implementation Producer implementation Jan 14, 2025
@Nayrode Nayrode changed the title Producer implementation Draft: Producer implementation Jan 14, 2025
@Nayrode Nayrode marked this pull request as draft January 14, 2025 13:12
@Nayrode Nayrode changed the title Draft: Producer implementation Producer implementation Jan 14, 2025
@Nayrode Nayrode marked this pull request as ready for review January 14, 2025 15:14
@Nayrode Nayrode force-pushed the 6-producer-implementation branch 6 times, most recently from 457948a to fea436a Compare January 14, 2025 15:54
Razano26
Razano26 requested changes Jan 15, 2025
View reviewed changes
.github/workflows/rust.yml Outdated Show resolved Hide resolved
.gitignore Show resolved Hide resolved
README.md Show resolved Hide resolved
producer/README.md Show resolved Hide resolved
@Nayrode Nayrode force-pushed the 6-producer-implementation branch from ed86675 to 6c0d140 Compare January 15, 2025 08:54
Nayrode added 12 commits January 15, 2025 09:57
@Nayrode
feat: doesn't work but it's a start
f99d7d3
@Nayrode
feat:database writes working
4fded98
@Nayrode
feat: improving logic between ressources
9753826
@Nayrode
feat: refactor to be able to use common interfaces
ed649b2
@Nayrode
feat: refactor to be able to use common interfaces
8cafea8
@Nayrode
feat: add async traits for database operations in common and producer…
9a2cbe3 
… modules
@Nayrode
feat: update .gitignore to exclude .sqlx files
0dcbe74
@Nayrode
feat: update project structure and enhance producer documentation
9cd334f
@Nayrode
feat: integrate rdkafka for command processing and update dependencies
98369b7
@Nayrode
feat: add clap for command-line argument parsing and enhance database…
c3b5ad6 
… seeding functionality
@Nayrode
feat: update random product generation to use specific fake data and …
5c1bdcc 
…random price range
@Nayrode
feat: update Cargo.toml to use resolver version 2 and clean up unused…
d5208aa 
… imports in main.rs
@Nayrode Nayrode force-pushed the 6-producer-implementation branch from 6c0d140 to 8546426 Compare January 15, 2025 08:58
@Nayrode Nayrode requested a review from Razano26 January 15, 2025 08:59
Nayrode
Nayrode commented Jan 15, 2025
View reviewed changes
.gitignore Show resolved Hide resolved
@dorian-grst dorian-grst added this pull request to the merge queue Jan 15, 2025
Merged via the queue into main with commit 25dc333 Jan 15, 2025
1 check passed
@dorian-grst dorian-grst deleted the 6-producer-implementation branch January 15, 2025 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dorian-grst dorian-grst dorian-grst approved these changes

@Razano26 Razano26 Razano26 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.

Producer Implementation
3 participants
@Nayrode @Razano26 @dorian-grst