Log occurred exception even if it doesn't have cause

[JuliaIskra]

I have a problem in my application where it cannot reconnect to Nakadi and I can't figure out why, because exception is not logged, apparently, because the occurring exception doesn't have a cause:

Reconnecting failed, retry again in 10 seconds, SubscriptionId: <uuid>'

[mdedetrich]

I don't think this will have the intended effect because it will just delegate to def info(message: String, args: Any*) and since args is just Any it will only print a reference/toString of e and not the staktrace (which is what I believe you want).

You need to either call def info(marker: Marker, message: String, cause: Throwable) or def warn(message: String, cause: Throwable)

[JuliaIskra]

I have the following issue:

my application connects to nakadi and reads events successfully several times:

but after a while the stream is disconnected by the server due to timeout (as above), but it cannot reconnect again:

And this lasts until I restart the application. As you can see there is no indication to what's the reason of the reconnection failure. That's why I want to add this logging.

Nakadi supporters told me that server responds with 401 for my app attempts to reconnect. I was using 0.9.2 kanadi lib. @mdedetrich @itachi3 @pascalh @gchudnov do you have any ideas why this might happen?

I plan to update to the latest version and hope to get this PR merged and released in version 0.11.1 to see what's really the root cause.

Could you please tell me when you can release a new version with this PR's changes?

[mdedetrich]

So I can approve the PR but I can't really release it (I don't actually work at Zalando anymore and the release is done manually on someones machine)

[gchudnov]

@JuliaIskra
Hi, wondering if you have something else around in logs?

[gchudnov]

let me check, will merge and release it soon

[gchudnov]

@mdedetrich
will merge if no objections

[mdedetrich]

@mdedetrich will merge if no objections

No objections from me, ill approve it just to make it official

[mdedetrich]

lgtm

[gchudnov]

Thank you,
merged

[gchudnov]

I'll release it in 0.11.1 version, given that @JuliaIskra is using akka, and later in pekko as well (0.20.1).

[gchudnov]

released 0.11.1 to sonatype for scala 2.12 and 2.13

https://github.com/zalando-nakadi/kanadi/releases

[JuliaIskra]

thank you so much!