Skip to content

feat(routing): Enable volume split for dynamic routing #6662

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
Merged
merged 46 commits into from
Dec 5, 2024
Merged

feat(routing): Enable volume split for dynamic routing #6662

merged 46 commits into from
Dec 5, 2024

Conversation

Sarthak1799
Copy link
Contributor

@Sarthak1799 Sarthak1799 commented Nov 26, 2024 •
edited
Loading

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Have added a new API for enabling volume split for dynamic routing service

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

curl --location --request POST 'http://localhost:8080/account/sarthak1/business_profile/pro_WlO7KCOeDRB8kWk7MSy1/dynamic_routing/set_volume_split?split=20' \
--header 'api-key: dev_iricUDSeFis6RzTGCcYkOt4wmuBcRfjbVbWXbOUO5996boAgztmtfTKN6HNqerjM'

Status in DB -
image

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

prajjwalkumar17 and others added 30 commits November 14, 2024 01:37
@prajjwalkumar17
feat(routing): elimination routing feature
d983f6c
@hyperswitch-bot
chore: run formatter
2f61113
@prajjwalkumar17
clippy lints
ecd8611
@hyperswitch-bot
chore: run formatter
62cd750
@prajjwalkumar17
add configs
6dd7e9c
@prajjwalkumar17
Merge branch 'feat_elimination_routing' of https://github.com/juspay/…
7ef5427 
…hyperswitch into feat_elimination_routing
@hyperswitch-bot
chore: run formatter
bf1aa32
@prajjwalkumar17
toggle api working for Elimination Routing
4c51dd4
@hyperswitch-bot
chore: run formatter
fe43bbe
@prajjwalkumar17
code refactors
21ce449
@prajjwalkumar17
merge
6f85320
@prajjwalkumar17
refactor old code
223c19d
@prajjwalkumar17
add disable feature
25a1ddf
@prajjwalkumar17
Merge branch 'main' into feat_elimination_routing
e6d86b8
@prajjwalkumar17
refactor the code
0f63a7f
@prajjwalkumar17
Merge branch 'feat_elimination_routing' of github.com:juspay/hyperswi…
ecbf598 
…tch into feat_elimination_routing
@prajjwalkumar17
refactor code
06db0e1
@hyperswitch-bot
chore: run formatter
8138814
@prajjwalkumar17
minor refactors to make code work
9d015f9
@prajjwalkumar17
final refactors
e1cf154
@hyperswitch-bot
chore: run formatter
f464b63
@Sarthak1799
feat: Introduce Volume split for dyanmo
2d3eb78
@prajjwalkumar17
fix openapi
30f4fa8
@Sarthak1799
Merge branch 'feat_elimination_routing' of github.com:juspay/hyperswi…
e8acceb 
…tch into dynamo-volume-split
@prajjwalkumar17
Merge branch 'feat_elimination_routing' of https://github.com/juspay/…
bb9262a 
…hyperswitch into feat_elimination_routing
@hyperswitch-bot
docs(openapi): re-generate OpenAPI specification
1f608b0
@prajjwalkumar17
fix spell checks
bd62f26
@prajjwalkumar17
Merge branch 'feat_elimination_routing' of https://github.com/juspay/…
8de6d47 
…hyperswitch into feat_elimination_routing
@prajjwalkumar17
add cache for elimination routing
31eded0
@prajjwalkumar17
minor refactors
9116e09
@hyperswitch-bot hyperswitch-bot bot removed M-database-changes Metadata: This PR involves database schema changes M-api-contract-changes Metadata: This PR involves API contract changes labels Nov 27, 2024
Base automatically changed from feat_elimination_routing to main December 2, 2024 15:10
@hyperswitch-bot hyperswitch-bot bot added the M-api-contract-changes Metadata: This PR involves API contract changes label Dec 2, 2024
@hyperswitch-bot hyperswitch-bot bot removed the M-api-contract-changes Metadata: This PR involves API contract changes label Dec 3, 2024
Chethan-rao
Chethan-rao suggested changes Dec 3, 2024
View reviewed changes
Chethan-rao
Chethan-rao previously approved these changes Dec 4, 2024
View reviewed changes
prajjwalkumar17
prajjwalkumar17 previously approved these changes Dec 4, 2024
View reviewed changes
Chethan-rao
Chethan-rao previously approved these changes Dec 4, 2024
View reviewed changes
prajjwalkumar17
prajjwalkumar17 previously approved these changes Dec 4, 2024
View reviewed changes
tsdk02
tsdk02 previously approved these changes Dec 4, 2024
View reviewed changes
SanchithHegde
SanchithHegde reviewed Dec 4, 2024
View reviewed changes
@likhinbopanna likhinbopanna added this pull request to the merge queue Dec 5, 2024
Merged via the queue into main with commit 03b936a Dec 5, 2024
16 of 18 checks passed
@likhinbopanna likhinbopanna deleted the dynamo-volume-split branch December 5, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SanchithHegde SanchithHegde SanchithHegde approved these changes

@tsdk02 tsdk02 tsdk02 approved these changes

@prajjwalkumar17 prajjwalkumar17 prajjwalkumar17 left review comments

+1 more reviewer

@Chethan-rao Chethan-rao Chethan-rao approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@Sarthak1799 Sarthak1799

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Volume Split for Intelligent Routing
6 participants
@Sarthak1799 @SanchithHegde @prajjwalkumar17 @Chethan-rao @tsdk02 @likhinbopanna