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

add test for exception during timeboundary #1303

Merged
merged 1 commit into from
Apr 23, 2015

Conversation

nishantmonu51
Copy link
Member

Add Test for exception during time boundary query where results can be
represented as integers
Stack trace of exception -
java.lang.IllegalArgumentException: No instant converter found for type: java.lang.Integer
at org.joda.time.convert.ConverterManager.getInstantConverter(ConverterManager.java:165) ~[druid-selfcontained-0c734c6.jar:0c734c6]
at org.joda.time.base.BaseDateTime.(BaseDateTime.java:173) ~[druid-selfcontained-0c734c6.jar:0c734c6]
at org.joda.time.DateTime.(DateTime.java:257) ~[druid-selfcontained-0c734c6.jar:0c734c6]
at io.druid.query.timeboundary.TimeBoundaryQueryQueryToolChest$5$2.apply(TimeBoundaryQueryQueryToolChest.java:189) ~[druid-selfcontained-0c734c6.jar:0c734c6]

@fjy
Copy link
Contributor

fjy commented Apr 22, 2015

@nishantmonu51 I think xavier made a similar fix, can you update your branch to be mergable?

@xvrl
Copy link
Member

xvrl commented Apr 22, 2015

@nishantmonu51 I fixed that issue as well as a bunch more in order query types as part of #1304.
We can add you tests though if you want.

Fix Exception during time boundary query where results can be
represented as integers
@nishantmonu51 nishantmonu51 changed the title fix exception during timeboundary add test for exception during timeboundary Apr 23, 2015
xvrl added a commit that referenced this pull request Apr 23, 2015
add test for exception during timeboundary
@xvrl xvrl merged commit 7081c83 into apache:master Apr 23, 2015
@xvrl xvrl deleted the fix-timeboundary-caching branch April 23, 2015 21:46
seoeun25 pushed a commit to seoeun25/incubator-druid that referenced this pull request Jan 10, 2020
seoeun25 pushed a commit to seoeun25/incubator-druid that referenced this pull request Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants