From: <tho...@us...> - 2011-01-15 22:49:46
|
Revision: 4105 http://bigdata.svn.sourceforge.net/bigdata/?rev=4105&view=rev Author: thompsonbry Date: 2011-01-15 22:49:40 +0000 (Sat, 15 Jan 2011) Log Message: ----------- Modified the thrown exception from IllegalArgumentException in RWStore#readData() to RuntimeException. This reflects the fact that the cause could be anything, including a ClosedByInterruptException which is most certainly not the result of an illegal argument. Modified Paths: -------------- branches/QUADS_QUERY_BRANCH/bigdata/src/java/com/bigdata/rwstore/RWStore.java Modified: branches/QUADS_QUERY_BRANCH/bigdata/src/java/com/bigdata/rwstore/RWStore.java =================================================================== --- branches/QUADS_QUERY_BRANCH/bigdata/src/java/com/bigdata/rwstore/RWStore.java 2011-01-15 22:21:02 UTC (rev 4104) +++ branches/QUADS_QUERY_BRANCH/bigdata/src/java/com/bigdata/rwstore/RWStore.java 2011-01-15 22:49:40 UTC (rev 4105) @@ -1435,12 +1435,14 @@ /* * Note: ClosedByInterruptException can be thrown out of * FileChannelUtility.readAll(), typically because the LIMIT on - * a query was satisified, but we do not want to log that as an + * a query was satisfied, but we do not want to log that as an * error. */ // log.error(e,e); - throw new IllegalArgumentException("Unable to read data", e); +// throw new IllegalArgumentException("Unable to read data: "+e, e); + throw new RuntimeException(e); + } } finally { readLock.unlock(); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |