-
Notifications
You must be signed in to change notification settings - Fork 25.2k
TSDB: Expand _id on version conflict #84957
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TSDB: Expand _id on version conflict #84957
Conversation
I'd love to get folk's opinion on this one, especially @henningandersen. The new error message looks like:
|
This adds the `_tsid` and `@timestamp` to the version conflict error message in tsdb mode. I'm trying to debug a version confict problem with tsdb and think it'd be useful to get this information rather than the generated `_id`.
server/src/main/java/org/elasticsearch/index/mapper/TsidExtractingIdFieldMapper.java
Outdated
Show resolved
Hide resolved
Pinging @elastic/es-analytics-geo (Team:Analytics) |
Hi @nik9000, I've created a changelog YAML for you. |
b7475a7
to
9fc081d
Compare
@henningandersen I think this one is ready for you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to change the message format a bit, otherwise this looks good to me.
rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/tsdb/25_id_generation.yml
Outdated
Show resolved
Hide resolved
run elasticsearch-ci/part-1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Thanks folks! |
This adds the
_tsid
and@timestamp
to the version conflict errormessage in tsdb mode. I'm trying to debug a version confict problem with
tsdb and think it'd be useful to get this information rather than the
generated
_id
.