Jesse Szwedko

Results 542 comments of Jesse Szwedko

Yeah, that is an unfortunate gap. I do see some discussion around that in https://github.com/BurntSushi/ripgrep/pull/765 From there I found a link to https://github.com/gilnaa/globwalk which might be another candidate. More discussion...

Thanks for reviewing @bruceg . I realize I completely forgot about Windows here so I'll need to rework this a bit to abstract the signaling bits. I'll move this back...

> Moreover.. who cares about Windows? 🤯 If it's problematic to support Windows API right away, why not to merge this to fix 99,9% use-cases and add Windows support later?...

Moving this out of draft as I implemented non-graceful shutdown for Windows processes (to be improved later).

> I have a couple of suggestions below. I am also wondering if this will fail to terminate subprocesses that don't respond to the signal on Unix. That is, if...

@fanatid I think we should just make sure to satisfy the requirements of the license. In this case, given we _are_ redistributing the source including the licenses with the code...

After talking with Luke and Ana a bit, it seems like we are particularly interested in testing out specifically high cardinality and low cardinality events with transforms.

> @jszwedko Ping ! Apologies for the delay! We'll get this reviewed next week. The changes look straight-forward enough.

Hi @sproberts92 ! Is this ready for re-review? Feel free to tag when it is.