| Re: [eclipselink-dev] Index support |
|
Hello Dies, You can go ahead and make the necessary
changes. I can review the changes. -----Original
Message----- Hi James, It took a while and we ran into a few
other issues too but finally completed these tests. Here’s what was broken: Symfoware has its own CREATE INDEX
syntax, so I created DatabasePlatform#buildCreateIndex and overrode it in
SymfowarePlatform last year. You added a new class IndexDefinition
that has its own buildCreationWriter method, building the CREATE INDEX
statement in an incompatible way. Is there any particular reason you
didn’t just reuse/change DatabasePlatform#buildCreateIndex? Now we have code to create this
statement in both IndexDefinition#buildCreationWriter and
DatabasePlatform#buildCreateIndex. To clean this up and fix it for
Symfoware, I’d prefer IndexDefinition#buildCreationWriter to delegate to
DatabasePlatform#buildCreateIndex, changing its signature to accept the unique
flag. Similar for buildDeletionWriter. Are you okay with us making this change,
or do you want to do it yourself and we create a patch for SymfowarePlatform
that fixes any Symfoware specific issues afterwards? Thanks, Dies From:
eclipselink-dev-bounces@xxxxxxxxxxx
[mailto:eclipselink-dev-bounces@xxxxxxxxxxx] On
Behalf Of James Sutherland 0 DocumentEmail Hi Dies, I am refactoring the Symfoware auto index support under the
new index support. If you could run the tests on Symfoware when I am done
to ensure I did not break anything, that would be great. -----Original Message----- Hi James, As you may know I added some code half a year ago to have
indices generated for primary key columns for SymfowarePlatform. Indices on primary keys are mandatory and not created
automatically on this platform. I hope your design will take that into account (i.e. e.g. not
create duplicate indices for primary keys that have an @Index annotation). Thanks! Dies From:
eclipselink-dev-bounces@xxxxxxxxxxx
[mailto:eclipselink-dev-bounces@xxxxxxxxxxx] On
Behalf Of James Sutherland Normal
0 DocumentEmail I am working on adding database index DDL generation support for
2.2, See, |