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

Update Repo #1

Merged
merged 68 commits into from
Oct 18, 2018
Merged

Update Repo #1

merged 68 commits into from
Oct 18, 2018

Conversation

crweiner
Copy link
Owner

No description provided.

ntakouris and others added 30 commits October 9, 2018 18:42
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Add integration test to engine.

Signed-off-by: kuba-- <kuba@sourced.tech>
Adds Proper Apache License 2.0
Register COALESCE function. Update docs. Add integration test to engine.
Signed-off-by: Antonio Jesus Navarro Perez <antnavper@gmail.com>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Antonio Jesus Navarro Perez <antnavper@gmail.com>
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Add missing functions to documentation.
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Remove go-tip to do not use unnecessary Travis slots
sql: implement USE <table> statement
sql/analyzer: fix ambiguous table after natural join
The reorder projection rule recreates the projection with the table
columns and aliases. When an alias is used more than once in a
node expression, for example, a filter (a=1 or a=2) it is added twice.
This caused the second time to add nil as the aliases where deleted
after adding them to projections list.

Now the columns are added only once.

Signed-off-by: Javi Fontan <jfontan@gmail.com>
analyzer: fix crash with aliases used multiple times
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
…e name instead

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
 sql: resolve tables with database qualifier
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
…talog

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Signed-off-by: kuba-- <kuba@sourced.tech>
* Revert "pilosa: use different holder per db/table pair"

This reverts commit 144b824.

Keeps the test to load multiple indexes.

Signed-off-by: Javi Fontan <jfontan@gmail.com>

* travis: upgrade pilosa library for testing

Lets reopen holder.

Signed-off-by: Javi Fontan <jfontan@gmail.com>
…o pilosa-holder

Signed-off-by: kuba-- <kuba@sourced.tech>
…REATE TABLE

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
ajnavarro and others added 28 commits October 16, 2018 12:04
function: handle non number arguments in ceil, floor and round
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: kuba-- <kuba@sourced.tech>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
For numberT cast to the sqltype.
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Manuel Carmona <manu.carmona90@gmail.com>
sql/(plan,analyzer): track partitions of indexable tables too
*: add information_schema db with files and column_statistics tables
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
…r partition

Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
sql/plan: correctly count processed index rows globally instead of per partition
sql: plan: Add SHOW CREATE TABLE [Fix #406]
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
*: implement LOCK and UNLOCK of tables
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
sql: implement SHOW CREATE DATABASE statement
sql: add timing info to span traces
@codecov-io
Copy link

codecov-io commented Oct 18, 2018

Codecov Report

Merging #1 into master will decrease coverage by 0.32%.
The diff coverage is 58.07%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage    62.3%   61.97%   -0.33%     
==========================================
  Files         113      124      +11     
  Lines        9540    10284     +744     
==========================================
+ Hits         5944     6374     +430     
- Misses       3024     3301     +277     
- Partials      572      609      +37
Impacted Files Coverage Δ
sql/expression/function/registry.go 0% <ø> (ø) ⬆️
mem/table.go 69.78% <ø> (ø) ⬆️
sql/analyzer/analyzer.go 86.86% <ø> (ø) ⬆️
sql/plan/ddl.go 43.47% <0%> (ø) ⬆️
sql/plan/use.go 0% <0%> (ø)
sql/index/pilosa/index.go 71.8% <0%> (ø) ⬆️
sql/plan/insert.go 0% <0%> (ø) ⬆️
sql/unresolved_database.go 0% <0%> (ø) ⬆️
sql/analyzer/readonly.go 0% <0%> (ø) ⬆️
sql/functionregistry.go 23.63% <0%> (-2.37%) ⬇️
... and 47 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5af085...5077ae6. Read the comment docs.

@crweiner crweiner merged commit a1a9bda into crweiner:master Oct 18, 2018
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.

7 participants