-
Notifications
You must be signed in to change notification settings - Fork 25.8k
ILM: fix after rendering to xcontent #33282
Copy link
Copy link
Closed
Labels
:Data Management/ILM+SLMDO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead.DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead.
Description
we currently ignore the TimeValue string representation and use our own that forces it to be represented in seconds. This is not user friendly, since our time-scale is often way beyond seconds.
The offending code:
Line 131 in ba8d4eb
| builder.field(AFTER_FIELD.getPreferredName(), after.seconds() + "s"); // Need a better way to get a parsable format out here |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
:Data Management/ILM+SLMDO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead.DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead.
Type
Fields
Give feedbackNo fields configured for issues without a type.