kapilt / contentmirror

Automatically exported from code.google.com/p/contentmirror
1 stars 0 forks source link

Oracle reserved keywords #43

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. ./bin/instance run parts/productdistros/ContentMirror/ddl.py oracle >
~/schema.sql
2. Try to import the schema into oracle
3.

What is the expected output? What do you see instead?

I expect the tables to be created, but instead I get invalid column
identifier errors for the size field on the files table, and the uid field
on the content table.

What version of the product are you using? On what operating system?

RHEL4, Oracle 10g, ContetMirror 0.5.2b2

Original issue reported on code.google.com by dave.him...@gmail.com on 4 Jun 2009 at 9:01

GoogleCodeExporter commented 9 years ago
thanks for the report, just noticed firebird seems to have the same problem ( 
as well
as on the type field). i'll need to discuss this on list as its going to affect
backwards compatiblity. 

Original comment by kapilt@gmail.com on 4 Jun 2009 at 11:12

GoogleCodeExporter commented 9 years ago

Original comment by kapilt@gmail.com on 4 Jun 2009 at 11:12

GoogleCodeExporter commented 9 years ago

Original comment by kapilt@gmail.com on 22 Mar 2010 at 6:05

GoogleCodeExporter commented 9 years ago

Original comment by kapilt@gmail.com on 22 Mar 2010 at 10:00

GoogleCodeExporter commented 9 years ago
it might be best to add this to sqlalchemy directly as well, contentmirror 
basically 
looks at sqlalchemy's known reserved words for a dialect, and prefixes 
attributes 
accordingly. I'm going to fix the framework schema (size, uid, type), but for 
user 
defined at schemas, this needs to be dealt with at the sqlalchemy level.

Original comment by kapilt@gmail.com on 26 Mar 2010 at 11:53

GoogleCodeExporter commented 9 years ago
renamed the following fields for compatibility, content.uid -> 
content.content_uid, 
content.type -> content.object_type, file.size -> file.file_size

Original comment by kapilt@gmail.com on 27 Mar 2010 at 12:06