[
https://issues.apache.org/jira/browse/OODT-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13230838#comment-13230838
]
[email protected] commented on OODT-410:
----------------------------------------------------
bq. On 2012-03-15 00:35:26, brian Foster wrote:
bq. > Why can't DataSourceCatalog be extended and just modified per hypersonic
needs?... what are the differences exactly between DataSource and Hyersonic
versions of the Catalogs?
bq.
bq. Chris Mattmann wrote:
bq. Hey Brian, great question. The reality is that this catalog was hidden
away in the File Manager test suite inside of TestDataSourceCatalog. There was
something wiggy about the DataSourceCatalog code that Hypersonic SQL didn't
like so I had to extend DataSourceCatalog for internal testing since it made
sense to use Hypersonic SQL for that (could load up the DBMS in memory, etc.) I
know that there was like 1 method that used incompatible SQL syntax that
Hypersonic SQL didn't like. Then when I overrode that one method, I had to
override like 4-5 others to inject my subclass method into the call chain for
that which is what you see in the patch. It seems that DataSourceCatalog has
since evolved and it's entirely possible that it would support Hypersonic SQL
now, so here's what I see as 2 options:
bq.
bq. 1. we try and simply remove Hypersonic SQL catalog period, and then
see if the TestDataSourceCatalog passes. If it does, this issue is moot and I
will close this ReviewBoard and the issue.
bq.
bq. 2. If #1 doesn't succeed, and suggests changes, we could:
bq. a) flow those changes into DataSourceCatalog -- someone would have
to have time to do that :) It also could happen later; or
bq. b) suggest that since this catalog exists and we know it works,
temporarily in 0.4 and ongoing until someone refactors or improves
DataSourceCatalog ship the HypersonicSQLFriendlyCatalog for folks who want to
use it.
bq.
bq. Sound right? Thoughts?
i'd be cool with you checking it in for now with a TODO at the top of the class
and maybe a JIRA issue to merge the 2 at some point
- brian
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4358/#review5978
-----------------------------------------------------------
On 2012-03-14 23:40:55, Chris Mattmann wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/4358/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-03-14 23:40:55)
bq.
bq.
bq. Review request for oodt, brian Foster, Ricky Nguyen, Paul Ramirez, and
Thomas Bennett.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. DataSourceCatalog compatible with HypersonicSQL
bq.
bq.
bq. This addresses bug OODT-410.
bq. https://issues.apache.org/jira/browse/OODT-410
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/HsqlDbFriendlyDataSourceCatalog.java
PRE-CREATION
bq.
./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/HsqlDbFriendlyDataSourceCatalogFactory.java
PRE-CREATION
bq.
./trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/catalog/TestDataSourceCatalog.java
1294690
bq.
bq. Diff: https://reviews.apache.org/r/4358/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq.
bq. Thanks,
bq.
bq. Chris
bq.
bq.
> DataSourceCatalog compatible with HypersonicSQL
> -----------------------------------------------
>
> Key: OODT-410
> URL: https://issues.apache.org/jira/browse/OODT-410
> Project: OODT
> Issue Type: New Feature
> Components: file manager
> Reporter: Chris A. Mattmann
> Assignee: Chris A. Mattmann
> Fix For: 0.4
>
>
> This patch will provide a customizable extension of the DataSourceCatalog for
> use with HypersonicSQL DB. All of the existing DataSourceCatalog
> filemgr.properties config options will be reused.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
