Skip to content

Fixes for [OPTIQ-332] and [OPTIQ-333] - #3

Closed
jacques-n wants to merge 4 commits into
apache:masterfrom
jacques-n:optiq_0_9_wip
Closed

Fixes for [OPTIQ-332] and [OPTIQ-333]#3
jacques-n wants to merge 4 commits into
apache:masterfrom
jacques-n:optiq_0_9_wip

Conversation

@jacques-n

Copy link
Copy Markdown
Contributor

No description provided.

@enricobacis

Copy link
Copy Markdown

I used only the commit f12bb03 on my fork but it works as expected.

@asfgit
asfgit force-pushed the master branch 2 times, most recently from fe91d46 to 2712d7d Compare January 13, 2016 16:45
@jacques-n

Copy link
Copy Markdown
Contributor Author

was done as part of other issues.

@jacques-n jacques-n closed this Oct 8, 2016
zhumayun pushed a commit to zhumayun/calcite that referenced this pull request May 22, 2017
# This is the 1st commit message:
Added thetaSketch data type to DruidTypes

# This is the commit message apache#2:

Fixed code to comply with checkstyle

# This is the commit message apache#3:

Implemented thetaSketch and hyperUnique DruidTypes by adding an extra map to hold column type information

# This is the commit message apache#4:

bug fix: used fieldName as key, rather than output name

# This is the commit message apache#5:

Fixed test errors

# This is the commit message apache#6:

Fixed DruidTableFactory to include the proper druid types. (Fixes Integration test issues as well)
zhumayun pushed a commit to zhumayun/calcite that referenced this pull request May 22, 2017
# This is the 1st commit message:
no message

# This is the commit message apache#2:

Added test case to check for COUNT aggregate without DISTINCT for metrics of type hyperUnique

# This is the commit message apache#3:

Changed comment format

# This is the commit message apache#4:

added thetaSketch metric to food mart model definition so that thetaSketches can be tested

# This is the commit message apache#5:

Updated old test cases to take into account new thetaSketch metric (user_unique) for foodmart
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