drop temporary CT tables

[oschulte]

could use a general drop procedure

• this would save a lot of space
• also make the output easier to read

• change , to _ in Rchains. See https://github.com/sfu-cl-lab/FactorBase/blob/master/src/lattice/short_rnid_LatticeGenerator.java
• try dropping tables. Should work now.

[oschulte]

Hi Zhao @JanyQZ you're right I'm missing details. Not sure what happened. Here's the kind of procedure we need.

KeepTablesOnly(connection,dbname,list of tablenames)
Inputs:

• connection object connection
• name of database schema dbname
• list of tablenames

Output: all tables in dbname are dropped except for the tables listed tablenames

Example:

• dbname = unielwin_ct
• tablesname = a_ct

The only table left in unielwin_ct should be a_ct

[oschulte]

@JanyQZ Hi Jany, were you able to solve this problem? Can you please confirm that you are set up to get these messages from github?

[JanyQZ]

@oschulte Hi, Oliver, I am now working at drop_tmpTable branch. And I added a new class which named KeepTablesOnly.java. The mainly function looks like Drop_tmpTables(Connection con,String dbname,String tablename). The function is currently available to keep only one table given by the input.
What I am thinking is that we can add a new variable to indicate the number of reserved tables and change tablename into a array, which may looks like Drop_tmpTables(Connection con,String dbname,String[] tablesname). We can talk about this issue later :)

[oschulte]

@JanyQZ sounds good! I agree with having an array of tables.

[JanyQZ]

@oschulte Hi Oliver, I found that we couldn't drop the table with a comma name, such as "a,b_counts", cause mysql automatically splits the sql query based on commas assuming they are separate tables and returns an error message which like "Unknown table x,xxx". I think we really need to standardize the tables names and not using commas.

[oschulte]

Hi Jany, changing table names is not a bad idea but a fair amount of work. Have you tried escaping the table name with apostrophes? E.g.

drop table a,b_counts
not
drop table a,b_counts
?

[JanyQZ]

@oschulte Hi, Oliver, I've already tried both of the command, it didn't work.
Actually, it's very strange to add a comma into a table name. We should not do that at the very beginning.

[oschulte]

let's discuss with Vidhi

[JanyQZ]

@oschulte HI, Oliver, I found a way to solve this problem. We can wrap the table name with the symbol ``, such as a,b_counts, it will indicate entire string is one table name. I will fix this function as soon as possible and merge into the project.

[oschulte]

okay great. Can we drop tables now?

[JanyQZ]

Of course we can! Also you can see all the commit changes in pull request. It works well.

[oschulte]

Let's review this and see if we need to add some documentation

[oschulte]

Jany posted the drop table code