Skip to content

Conversation

@MarioAlexandruDan
Copy link
Contributor

@MarioAlexandruDan MarioAlexandruDan commented Dec 3, 2025

mustache code generation for java
HttpRequester modification for WithHTTPInfo methods

🧭 What and Why

🎟 JIRA Ticket API-135:
https://algolia.atlassian.net/browse/API-135

Changes included:

  • List changes

🧪 Test

mustache code generation for SearchClient
HttpRequester modification for WithHTTPInfo methods
@MarioAlexandruDan MarioAlexandruDan requested a review from a team as a code owner December 3, 2025 12:12
@algolia-bot
Copy link
Collaborator

algolia-bot commented Dec 3, 2025

✔️ Code generated!

Name Link
🪓 Triggered by 3e90f905985711faba8b0ece91d96dd1d50ce583
🍃 Generated commit f9aee6773a23f2d3d4ee47c7a6eac3a1ff62d8e0
🌲 Generated branch generated/mario-algolia
📊 Benchmark results

Benchmarks performed on the method using a mock server, the results might not reflect the real-world performance.

Language Req/s
java 1182

@MarioAlexandruDan MarioAlexandruDan marked this pull request as draft December 3, 2025 12:12
@MarioAlexandruDan MarioAlexandruDan changed the title API-135 feat(java): Add WithHTTPInfo method derivatives [API-135] Dec 3, 2025
@MarioAlexandruDan MarioAlexandruDan marked this pull request as ready for review December 3, 2025 13:21
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

congrats on your first pr!!!!!!!!!!!!!!

MarioAlexandruDan and others added 3 commits December 3, 2025 15:29
…com/algolia/internal/HttpRequester.java

Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

nice, too bad we can't reuse the WithHTTPInfo methods in the non WithHTTPInfo ones, but looks clean!!

Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

can you remove the [API-135] from the PR title? We only put them in the body, but the title will be added to the changelog and customer don't need this kind of infos

@MarioAlexandruDan MarioAlexandruDan changed the title feat(java): Add WithHTTPInfo method derivatives [API-135] feat(java): Add WithHTTPInfo method derivatives Dec 3, 2025
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

perfect, congrats on your first PR merged!

@MarioAlexandruDan MarioAlexandruDan merged commit a59c5ef into main Dec 3, 2025
19 checks passed
@MarioAlexandruDan MarioAlexandruDan deleted the mario-algolia branch December 3, 2025 14:10
algolia-bot added a commit that referenced this pull request Dec 3, 2025
…skip ci]

Co-authored-by: Mario-Alexandru Dan <marioalexandrudan@gmail.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
algolia-bot added a commit to algolia/algoliasearch-client-java that referenced this pull request Dec 3, 2025
algolia/api-clients-automation#5712

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Mario-Alexandru Dan <marioalexandrudan@gmail.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
MarioAlexandruDan added a commit that referenced this pull request Dec 4, 2025
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.

5 participants