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

GRAD2-2929: task is completed. #527

Merged
merged 13 commits into from
Sep 18, 2024
Merged

GRAD2-2929: task is completed. #527

merged 13 commits into from
Sep 18, 2024

Conversation

infstar
Copy link
Contributor

@infstar infstar commented Sep 12, 2024

GRAD2-2929: task is completed.

GRAD2-2929: initial commit
GRAD2-2929: task is complete.
Fixed the conflict file.
GRAD2-2929: initial commit
GRAD2-2929: task is complete.
Fixed the conflict file.
Fixed the broken unit tests.
More coverage to provide more test cases & also remove unused codes.
Clean up unused imports.
Clean up unused codes.
Copy link
Contributor

@arybakov-cgi arybakov-cgi left a comment

Choose a reason for hiding this comment

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

Please answer to the comments

return WebClient.builder().exchangeStrategies(ExchangeStrategies.builder()
.codecs(configurer -> configurer
.defaultCodecs()
.maxInMemorySize(100 * 1024 * 1024)) // 100 MB
Copy link
Contributor

Choose a reason for hiding this comment

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

Is 100M enough? We have 300M in other apis

return WebClient.builder().exchangeStrategies(ExchangeStrategies.builder()
.codecs(configurer -> configurer
.defaultCodecs()
.maxInMemorySize(100 * 1024 * 1024)) // 100 MB
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here

Clean up more comments.
Copy link

@infstar infstar merged commit aef7df2 into grad-release Sep 18, 2024
5 checks passed
@infstar infstar deleted the feature/GRAD2-2929 branch September 18, 2024 21:09
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