verdict-project / verdict

Interactive-Speed Analytics: 200x Faster, 200x Fewer Cluster Resources, Approximate Query Processing
http://verdictdb.org
Apache License 2.0
248 stars 66 forks source link

Simpler "drop scramble" #308

Closed pyongjoo closed 5 years ago

pyongjoo commented 5 years ago

I noticed that its syntax includes the original table; however, since the scramble itself is a unique identifier, I don't we need the original table name.

I think it is good to have an extra checking, but just in case the user simply issues "drop scramble some_schema.scramble_table", we can drop the table.

pyongjoo commented 5 years ago

Can you also make scrambling queries use the default schema if available?

I just noticed that if the schema name is not explicitly specified (after a 'use' query), a syntax error is thrown.