-
Notifications
You must be signed in to change notification settings - Fork 25.3k
Fix DateMathExpressionResolverTests tests #37037 #37059
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -201,7 +201,7 @@ October 28, 2015 | |
they did before the upgrade. For example if `watcher.dynamic_indices.time_zone` | ||
setting was set to `+01:00` and a watch has the following index name | ||
`<logstash-{now/d}>` then after the upgrade you need to update this watch to | ||
use the following index name `<logstash-{now/d{YYYY.MM.dd|+01:00}}>`. | ||
use the following index name `<logstash-{now/d{yyyy.MM.dd|+01:00}}>`. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. these are old release notes, that might be outdated in other ways as well, I am not sure if it is worth updating? |
||
|
||
.New Features | ||
* Added new <<actions-hipchat, HipChat Action>> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -70,7 +70,7 @@ public abstract class Exporter implements AutoCloseable { | |
Setting.affixKeySetting("xpack.monitoring.exporters.","index.name.time_format", | ||
key -> Setting.simpleString(key, Property.Dynamic, Property.NodeScope)); | ||
|
||
private static final String INDEX_FORMAT = "YYYY.MM.dd"; | ||
private static final String INDEX_FORMAT = "yyyy.MM.dd"; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. the monitoring side of things is fixed in #35809 |
||
|
||
protected final Config config; | ||
|
||
|
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 am confused by reading this. Shouldnt one be in capital letters and one not in order for this to work?
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.
Java time cares significantly about
Y
vsy
. Afaict, Joda time only usesY
vsy
to exclude0
and negative numbers (actually, the documentation I can find differs about whether or not0
is a valid value, but...).That said,
y
in joda is the "year" w/o the constraint, which is a closer map to Javay
.