Suyog Rao

Results 14 comments of Suyog Rao

@Cherepushko can you rebase please? Also, can you add in the description and code comments we are removing invalid fields (with only dot as name) so Elasticsearch doesn't throw exception...

Also left a comment for adding unit test for the method that removes dots from k8s logs

@prathapreddy123 is it better to place this inside of google cloud to elasticsearch folder?

@prathapreddy123 can we get a review please?

LGTM from Elastic side

@prathapreddy123 any reason this is still open?

Just as an FYI when we get to this. The original ruby (and C!) implementation from @jordansissel is here https://github.com/jordansissel/ruby-grok. We should port all the awesome tests from that library...