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

File input #24

Closed
wants to merge 1 commit into from
Closed

File input #24

wants to merge 1 commit into from

Conversation

Trojan295
Copy link
Contributor

Fixes #20.

The file input:

  • supports tail-like reading for existing files and dynamic recognition of new files (via inotify)
  • handles multiple file inputs (src path can be glob compatible string with wildcards)

@Trojan295
Copy link
Contributor Author

I see, that the CI is failing on FreeBSD, because of inotify, which is a Linux kernel feature.

Looks it can be tricky to support Linux and FreeBSD.

@jedisct1
Copy link
Contributor

jedisct1 commented Mar 8, 2018

Errr.. yes, don't break portability please...

@jedisct1
Copy link
Contributor

jedisct1 commented Mar 8, 2018

Maybe use https://github.com/passcod/notify ?

@Trojan295
Copy link
Contributor Author

This passcod/notify looks promising!

@Trojan295
Copy link
Contributor Author

There is some problem with passcod/notify also. I created a issue there notify-rs/notify#155. I would say this PR is put on hold, till notify v5 is released.

@mirwan
Copy link

mirwan commented Sep 6, 2018

@Trojan295
Any update on this one? seems like notify-rs/notify#155 is closed

@Trojan295
Copy link
Contributor Author

I'll try to look at this today.

@Trojan295 Trojan295 force-pushed the file-input branch 2 times, most recently from a1c54c7 to f91f52a Compare September 8, 2018 12:16
@Trojan295
Copy link
Contributor Author

Ok, CI is green, so I believe it can be reviewed. @jedisct1

@Trojan295 Trojan295 closed this Oct 2, 2018
@Trojan295 Trojan295 reopened this Oct 2, 2018
crisidev added a commit to amazon-archives/flowgger-hold that referenced this pull request Jul 19, 2019
@crisidev
Copy link
Contributor

We merged this PR with small changes during the transition to awslabs. Closing.

@crisidev crisidev closed this Sep 17, 2019
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.

File input
4 participants