Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 344669

Summary: [DB] Crash detection fails when using Oracle JDBC
Product: [Modeling] EMF Reporter: Erdal Karaca <erdal.karaca.de>
Component: cdo.dbAssignee: Eike Stepper <stepper>
Status: CLOSED FIXED QA Contact: Eike Stepper <stepper>
Severity: normal    
Priority: P3 CC: robert.blust, stepper
Version: 4.2   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on:    
Bug Blocks: 270428    
Attachments:
Description Flags
applied the pattern described in the bug report
none
the previous patch contained too much
none
Some more locations to apply that pattern none

Description Erdal Karaca CLA 2011-05-04 04:50:13 EDT
In most cases the Oracle JDBC driver returns BigDecimal or BigInteger when you expect a Long/Integer/Double.
In such cases I have been using this pattern:

((Number)numericValueReturnedByOracleResultSet).longValue()|intValue()|...Value()
Comment 1 Erdal Karaca CLA 2011-05-04 04:52:06 EDT
Created attachment 194680 [details]
applied the pattern described in the bug report
Comment 2 Erdal Karaca CLA 2011-05-04 04:54:07 EDT
Created attachment 194681 [details]
the previous patch contained too much

this patch only affects MetaDataManager
Comment 3 Erdal Karaca CLA 2011-05-05 09:23:45 EDT
Created attachment 194829 [details]
Some more locations to apply that pattern
Comment 4 Eike Stepper CLA 2011-06-23 03:58:02 EDT
Moving all open enhancement requests to 4.1
Comment 5 Eike Stepper CLA 2012-08-14 22:52:02 EDT
Moving all open issues to 4.2. Open bugs can be ported to 4.1 maintenance after they've been fixed in master.
Comment 6 Eike Stepper CLA 2012-11-01 23:49:27 EDT
I think I already fixed most of those. The only place with wrong casts I could find now is HorizontalBranchingMappingStrategyWithRanges.ImportListHandler.handleRow()
Comment 7 Eike Stepper CLA 2012-11-01 23:50:16 EDT
commit 7236e96ba40506d6bf09e27517f8edf079bcadd0
Comment 8 Eike Stepper CLA 2013-06-27 03:31:59 EDT
Available in R20130613-1157 (4.2)