app/vlinsert/opentelemetry: fix field collisions after flattening#1379
app/vlinsert/opentelemetry: fix field collisions after flattening#1379yagmurcicekdagi wants to merge 1 commit into
Conversation
|
Hi, Can you review this pr ? @func25? I've fixed the field name collisions, haven't implemented the query/merge path duplicate name issue yet. Let me know what you think so I can adjust the code. |
d6d6440 to
b8779ba
Compare
|
Hey, thank you for your interest in this issue.
I'd recommend starting with a few smaller issues first to get familiar with the logic, the code conventions and the contribution guidelines. As I skimmed through it, this PR appears to be a low-quality AI-generated contribution. |
|
I did use AI to help me locate the issue and give me skeleton to work on, but if you mean the abstractions that I put , that was from me unfortunately -which breaks the contribution guidelines Could you maybe guide me a bit to how to make this code better quality? I'd still love to work on getting this to better quality code, but if you don't want that, that's totally understandable as well Thanks for the reply @func25 |
func25
left a comment
There was a problem hiding this comment.
No problem. I saw some improvements/bug fixes in this PR itself, it's not entirely AI-generated slop, so haven't closed it. Please see below comments.
Using AI to enhance quality and make small improvements is encouraged, but not for this kind of task (yet).
|
Hey @yagmurcicekdagi, any updates? |
Signed-off-by: Yağmur Çiçekdağı <ygmcicekdagi@gmail.com>
b8779ba to
8ced0d1
Compare
|
Hey, I've updated the code @func25 |
Changes
Fixes: #1371
-opentelemetry.enableFieldPrefixesflag (disabled by default). When enabled, fields are prefixedby their source before flattening:
Added tests covering prefix mode for all field sources and verify generated fields
(trace_id, span_id, severity_number, severity_text) remain unprefixed.
Updated docs/victorialogs/data-ingestion/opentelemetry.md with a Field prefixes
section and victoria_logs_common_flags.md with the new flag.