Skip to content
This repository has been archived by the owner on Apr 18, 2022. It is now read-only.

Avoid file locks for opened files #42

Closed
masonkmeyer opened this issue May 22, 2018 · 7 comments
Closed

Avoid file locks for opened files #42

masonkmeyer opened this issue May 22, 2018 · 7 comments

Comments

@masonkmeyer
Copy link
Contributor

The RollingFile sink creates a SharedFileSink when you pass the shared flag (serilog/serilog#1005).

Is it possible to add a setting to the KinesisFirehoseSinkOptions to pass a shared setting to the underlying RollingFileSink?

It looks like it could be set here:
https://github.com/serilog/serilog-sinks-amazonkinesis/blob/master/src/Serilog.Sinks.Amazon.Kinesis/Firehose/Sinks/DurableKinesisFirehoseSink.cs#L39

I would be happy to create a PR for this change.

@troylar
Copy link

troylar commented May 22, 2018

Go ahead, I'll take a look.

@masonkmeyer
Copy link
Contributor Author

This change would require an upgrade to some of the packages. It is dependant on #38 being merged (and then some additional package upgrades). I would still like to do it but I understand that it has an effect on the larger community.

@masonkmeyer
Copy link
Contributor Author

Nevermind. I was looking at the dev branch. I will make the change and get a PR for you.

@mattbeatty
Copy link

I tested @masonkmeyer's change and made a PR #45

@mattbeatty
Copy link

@troylar Let me know when you get a chance to check out that PR. 👍

@troylar troylar closed this as completed Jul 14, 2018
@troylar
Copy link

troylar commented Jul 14, 2018

Merged

@mattbeatty
Copy link

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants