Vasiliy Kulikov

Results 19 comments of Vasiliy Kulikov

Hi! Any update on this? I'd want to use the feature of line:col. I see that there is a merge conflict here. If there is no developer to update the...

@masatake , renamed also removed 'info' parsing as it is not very informative

@masatake hi! What should we do with this PR? Do you expect me to fix the conflict, or you want to e.g. split it into parts and merge the parts...

Yeah, no problem! Thx

@masatake OK, seems fair. I'll try to split it following the necessary patch "granulation".

@masatake friendly ping. Should I update this PR and show you for code review, or do you want to do it by yourself?

OK, I'll try it the next week.

Can you send us the minimal reproducer? pytest test would be ideal.

Tests are failing! Please look at userver/universal/src/formats/json/impl/types_impl.cpp (yes, a tricky one) how rapidjson's Value must be destroyed to avoid stack overflow.

We'll implement a full-blown opentelemetry implementation using OTLP and gRPC collector soon. So current implementation will be deprecated.