Daijiro Fukuda
Daijiro Fukuda
@yugeeklab The CI issue has been resolved. Sorry for the trouble. Could you please rebase this branch on the latest master?
Sorry for waiting. I will review this soon.
@yugeeklab Thanks for updating. I'm checking the CI failures.
Current CI failures have nothing to do with this PR. Sorry for the trouble again.
The CI issue has been resolved. So, could you please rebase this to the latest master? Sorry for the trouble again.
> We should not update `pos` like [this commit (bf73efe)](https://github.com/fluent/fluentd/commit/bf73efeea722376f30bccac0f10095fb6601939c). > ... > If updating `pos` like this commit, some data may be lost when `BufferOverflowError` occurs or when Fluentd...
> So, we need to consider only the following points. > > > For this feature, we need to take care of `@was_long_line` in particular. > > We need to...
@yugeeklab I have fixed the remaining points and pushed them to my tmp branch (the following 3 commits). Could you please check them? If there is no problem, I will...
@yugeeklab Yes! Thanks for checking it! I will push them.
@yugeeklab Sorry, I failed to push. Something wrong happens... I'm fixing it. Please wait...