Skip to content
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

Jump to bottom

Date parsing: Ensure parsing negative unix timestamps works #11478

Closed
spinscale opened this issue Jun 3, 2015 · 0 comments · Fixed by #11482
Closed

Date parsing: Ensure parsing negative unix timestamps works #11478

spinscale opened this issue Jun 3, 2015 · 0 comments · Fixed by #11482
Assignees
@spinscale
Labels
:Search/Mapping Index mappings, including merging and defining field types v2.0.0-beta1

Comments

@spinscale
Copy link
Contributor

A bug was introduced in the commit 01e8eaf - negative unix timestamps are not parsed correctly.

Sample failure at http://build-us-00.elastic.co/job/es_core_master_metal/9694/

Relates #5328
@spinscale spinscale self-assigned this Jun 3, 2015
spinscale added a commit to spinscale/elasticsearch that referenced this issue Jun 3, 2015
@spinscale
Dates: Allow for negative unix timestamps
9453039 
This fixes an issue to allow for negative unix timestamps.
It also fixes the default date field mapper to support epochs.

Fixes elastic#11478
@spinscale spinscale mentioned this issue Jun 3, 2015
Merged
spinscale added a commit to spinscale/elasticsearch that referenced this issue Jun 4, 2015
@spinscale
Dates: Allow for negative unix timestamps
8c2aa4b 
This fixes an issue to allow for negative unix timestamps.
It also fixes the default date field mapper to support epochs.

Fixes elastic#11478
spinscale added a commit to spinscale/elasticsearch that referenced this issue Jun 22, 2015
@spinscale
Dates: Allow for negative unix timestamps
38ddc81 
This fixes an issue to allow for negative unix timestamps.
An own printer for epochs instead of just having a parser has been added.
Added docs that only 10/13 length unix timestamps are supported
Added docs in upgrade documentation

Fixes elastic#11478
@clintongormley clintongormley added :Search/Mapping Index mappings, including merging and defining field types and removed :Dates labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spinscale spinscale

Labels
:Search/Mapping Index mappings, including merging and defining field types v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow for negative unix timestamps spinscale/elasticsearch
2 participants
@clintongormley @spinscale