Skip to content
This repository was archived by the owner on Jan 28, 2021. It is now read-only.

sql: plan: Add SHOW CREATE TABLE [Fix #406] #435

Merged
merged 15 commits into from
Oct 17, 2018

Conversation

ntakouris
Copy link
Contributor

@ntakouris ntakouris commented Oct 9, 2018

#406

Signed-off-by: Theodoros Zarkopafilis Ntakouris zarkopafilis@gmail.com

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
@ajnavarro ajnavarro requested review from erizocosmico, mcarmonaa and kuba-- and removed request for erizocosmico October 10, 2018 08:54
Copy link
Contributor Author

@ntakouris ntakouris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kuba-- The whitespace is not missed. Ill change the string() and the strings. Any ideas regarding the database on parser.go

…e name instead

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
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>
…REATE TABLE

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
@ajnavarro
Copy link
Contributor

@Zarkopafilis can you rebase please?

@erizocosmico
Copy link
Contributor

@Zarkopafilis CI is failing now, can you please take a look at it?

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
@ntakouris
Copy link
Contributor Author

I have no idea why this happened. Fixed.

@erizocosmico
Copy link
Contributor

Shall we merge this, @ajnavarro?

@ajnavarro
Copy link
Contributor

It is still failing @erizocosmico @Zarkopafilis

Signed-off-by: Theodoros Zarkopafilis Ntakouris <zarkopafilis@gmail.com>
@ntakouris
Copy link
Contributor Author

Excuse me, apparently something is off with my golang installation and eveything seemed fine. Rebase worked but the Catalog interface got changed a bit.

@erizocosmico
Copy link
Contributor

@ajnavarro now it's ok. It's failing because of the java thing, but the fix is on master anyway.

@ajnavarro ajnavarro merged commit b830c9a into src-d:master Oct 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy