EventTime msgpack ext type

[Jake Sperlazza]

Hello,

I'm using a cloudwatch output plugin. Consider this line in the code. When to_message pack is called on the time, it's using the normal to_msgpack method of EventTime, which only returns the seconds.

What's the proper way to get this to use the _ext version instead?

Thank you

[Mr. Fiber]

> What's the proper way to get this to use the _ext version instead?

The proper way is don't call to_msgpack in plugin side.

Default implementation handles EventTime correctly.

Open an issue is better.

Masahiro

--
You received this message because you are subscribed to the Google Groups "Fluentd Google Group" group.
To unsubscribe from this group and stop receiving emails from it, send an email to fluentd+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Jake Sperlazza]

Thank you for the response. After reading through some more code I think I found out what I needed to do.

I have opened an issue, and am just waiting on my PR to be reviewed.

On Monday, June 11, 2018 at 8:09:57 AM UTC-4, repeatedly wrote:

> What's the proper way to get this to use the _ext version instead?

The proper way is don't call to_msgpack in plugin side.

Default implementation handles EventTime correctly.

Open an issue is better.

Masahiro

On Sun, Jun 10, 2018 at 4:56 PM, Jake Sperlazza <supert...@gmail.com> wrote:

Hello,

I'm using a cloudwatch output plugin. Consider this line in the code. When to_message pack is called on the time, it's using the normal to_msgpack method of EventTime, which only returns the seconds.

What's the proper way to get this to use the _ext version instead?

Thank you

--
You received this message because you are subscribed to the Google Groups "Fluentd Google Group" group.

To unsubscribe from this group and stop receiving emails from it, send an email to fluentd+u...@googlegroups.com.