[jbosstools-dev] Re: JBIDE-2173 & JBIDE-2162 - performance improvement (for MySQL?)

[Max Rydahl Andersen]

>>> Did you create a patch for this ? Does all unittests passes ? I assume
> alot of logic
>>> has to change now that foreignkeys are in "reverse" order ?
>
> Yes I have created patch for this ->
> https://jira.jboss.org/jira/browse/JBIDE-2173 -> JBIDE-2173_20080630.zip.
> I added some info about process time before and after the patch.
> I compare unittests before patch and after - it had the same number of
> Runs/Errors/Failures in both cases.
> You could check the patch to find possible problems - as I expect it  
> should
> work -
> may be it's logic not so smooth - we could discuss it to find better
> solution.
> I'll compare all unittests once more for better confidence and report you
> about all problems.

Any news ?

>>> sorry, but I don't see what is obvious in that ?
>
> You could see this changes in JBIDE-2173_20080630.zip ->
> org.hibernate.cfg.reveng_patch.txt ->
>>>
> -				Table fkTable = dbs.getTable(fkSchema,
> fkCatalog, fkTableName);
> +				Table fkTable =
> dbs.getTable(quote(fkSchema), quote(fkCatalog), quote(fkTableName));
>>>
> I find this error in some unittest for quotes (when I tested
> JBIDE-2173_20080630 patch with unittestes) ->
> This is a way to add table into dbs:
>
> Table table = dbs.addTable(quote(getSchemaForModel(schemaName)),
> getCatalogForModel(catalogName), quote(tableName));
>
> You see - we add quotes for schema & table (if it's necessary to have  
> quotes
> added),
> but then we try to get table -> dbs.getTable we have fkSchema without  
> quotes
> - and get an error.
> In the case without my patch this situation was no reproducible.
>
>>> > I think the better way incorporate quotes process into
> DatabaseCollector.
>>> Why ?
>>> Please explain.
>
> My statement is disputable - " incorporate quotes process into
> DatabaseCollector " means
> We can use
>
>  Table fkTable = dbs.getTable(fkSchema, fkCatalog, fkTableName);
> &
>  Table table = dbs.addTable(getSchemaForModel(schemaName),
> getCatalogForModel(catalogName), tableName);
>
> And forget about quotes outside DatabaseCollector,

I can't see how we can forget about quotes outside db collector ?

> But seems we can't easy and beautiful transfer the function quote() from
> JDBCReader to DatabaseCollector offsprings.
> So statement is just disputable.

I'm sorry, but I don't understand your point. "is just disputable" ?

/max