python-sql: Add TRUNCATE

[ced, 2013-10-23 16:31:32]

Add a changelog entry

https://codereview.appspot.com/15740045/diff/30001/sql/tests/test_truncate.py
File sql/tests/test_truncate.py (right):

https://codereview.appspot.com/15740045/diff/30001/sql/tests/test_truncate.py...
sql/tests/test_truncate.py:48: finally:
Could also test params under this flavor

[nicoe, 2013-12-11 09:50:14]

https://codereview.appspot.com/15740045/diff/30001/sql/tests/test_truncate.py
File sql/tests/test_truncate.py (right):

https://codereview.appspot.com/15740045/diff/30001/sql/tests/test_truncate.py...
sql/tests/test_truncate.py:48: finally:
On 2013/10/23 16:31:32, ced wrote:
> Could also test params under this flavor

I don't understand

[ced, 2013-12-11 10:06:13]

https://codereview.appspot.com/15740045/diff/30001/sql/tests/test_truncate.py
File sql/tests/test_truncate.py (right):

https://codereview.appspot.com/15740045/diff/30001/sql/tests/test_truncate.py...
sql/tests/test_truncate.py:48: finally:
On 2013/12/11 09:50:14, nicoe wrote:
> On 2013/10/23 16:31:32, ced wrote:
> > Could also test params under this flavor
> 
> I don't understand

The params of the query

https://codereview.appspot.com/15740045/diff/100001/CHANGELOG
File CHANGELOG (right):

https://codereview.appspot.com/15740045/diff/100001/CHANGELOG#newcode1
CHANGELOG:1: * Add TRUNCATE
It is Truncate that is added.

[vvanderleun, 2014-04-10 10:21:49]

LGTM, please push.

[ced, 2014-04-10 10:34:46]

On 2014/04/10 10:21:49, vvanderleun wrote:
> LGTM, please push.

Not for me.

[ced, 2014-05-03 20:14:24]

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py
File sql/__init__.py (right):

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py#newcode634
sql/__init__.py:634: def __init__(self, table):
Truncate can take a list of tables as this command is about efficiency, it
should allow also to take a list of tables.

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py#newcode634
sql/__init__.py:634: def __init__(self, table):
There is also the RESTART/CONTINUE IDENTITY option which is part of the std.

[ced, 2014-05-05 10:00:45]

TRUNCATE is DDL while currently python-sql is DML only.
I don't know if we want to go to DDL support but at least it should be clearly
separated from DML.

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py
File sql/__init__.py (right):

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py#newcode634
sql/__init__.py:634: def __init__(self, table):
On 2014/05/03 20:14:24, ced wrote:
> Truncate can take a list of tables as this command is about efficiency, it
> should allow also to take a list of tables.

It seems to be PostgreSQL only.

[nicoe, 2014-05-06 16:17:35]

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py
File sql/__init__.py (right):

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py#newcode634
sql/__init__.py:634: def __init__(self, table):
The list of tables is not in the standard.

https://codereview.appspot.com/15740045/diff/120001/sql/__init__.py#newcode634
sql/__init__.py:634: def __init__(self, table):
On 2014/05/03 20:14:24, ced wrote:
> There is also the RESTART/CONTINUE IDENTITY option which is part of the std.

Done.

[nicoe, 2014-05-06 16:20:24]

I am not sure we should go on in this direction since the DDL queries seems too
much like one shot queries and they do not seems to require that much pythonic
bindings.