injae kim comments

Results 62 comments of


                                            injae kim

Add `StreamMessage.fromOutputStream()`

@ikhoon , @minwoox nim, really thanks for catching problem! I couldn't find the problem by myself 😅

Add `uri()` method to the `CentralDogma` client.

I'll try this issue! I'll create PR until `Feb 20, 2022`

Add `uri()` method to the `CentralDogma` client.

> We might add EndpointGroup instead of uri method if we decide to add Armeria dependency to centraldogma-client. oh.. thanks for sharing! I'll take a look #673 issue and add...

Fixed window counter implementation

Hi! I created PR https://github.com/resilience4j/resilience4j/pull/2100 to fix this issue. PTAL when you have some times~ thanks! 😃

Extend fetchSize support across JPA Item Readers

Nice work @baezzys !

Fixed DataBufferLimitException in WebFlux result in 413 instead of 500

origin issue: #32113 - Can you update PR description and commit message to include `Fixes #32113` ? - Can you update `copywright 2024` on top of each files?

Fixed DataBufferLimitException in WebFlux result in 413 instead of 500

oh sorry, I got it! actually PR author @mysend12 is my friend and I'm helping him to contribute(by pair programming!) so provide guideline by comment :) I'll don't do this~!...

A `DnsTimeoutException` should also set `true` to `ctx.isTimedOut()`.

Please assign this issue to me~! I'll make PR soon 🙇 thanks!

A `DnsTimeoutException` should also set `true` to `ctx.isTimedOut()`.

(Sorry for late, I'll make PR until 8/31 🙇)

Enhance DefaultUnhandledExceptionsReporter for Better Exception Reporting

Relate issue: https://github.com/line/armeria/issues/5150 I'll handle this issue with @coderclcl ~! 🙇