Strabon
changeset 1376:3f5a1c836f74
some comments and formatting
author | Babis Nikolaou <charnik@di.uoa.gr> |
---|---|
date | Tue Sep 16 23:14:57 2014 +0300 (2014-09-16) |
parents | d4105edb2022 |
children | 457324e89533 |
files | generaldb/src/main/java/org/openrdf/sail/generaldb/GeneralDBSqlTable.java generaldb/src/main/java/org/openrdf/sail/generaldb/GeneralDBValueFactory.java monetdb/src/main/java/org/openrdf/sail/monetdb/evaluation/MonetDBQueryBuilder.java postgis/src/main/java/org/openrdf/sail/postgis/evaluation/PostGISQueryBuilder.java |
line diff
1.1 --- a/generaldb/src/main/java/org/openrdf/sail/generaldb/GeneralDBSqlTable.java Tue Sep 16 23:11:40 2014 +0300 1.2 +++ b/generaldb/src/main/java/org/openrdf/sail/generaldb/GeneralDBSqlTable.java Tue Sep 16 23:14:57 2014 +0300 1.3 @@ -12,11 +12,11 @@ 1.4 /** 1.5 * Converts table names to lower-case and include the analyse optimisation. 1.6 * 1.7 - * @author James Leigh 1.8 - * 1.9 + * @author George Garbis <ggarbis@di.uoa.gr> 1.10 + * @author Manos Karpathiotakis <mk@di.uoa.gr> 1.11 */ 1.12 public abstract class GeneralDBSqlTable extends RdbmsTable { 1.13 - 1.14 + 1.15 public GeneralDBSqlTable(String name) { 1.16 super(name.toLowerCase()); 1.17 }
2.1 --- a/generaldb/src/main/java/org/openrdf/sail/generaldb/GeneralDBValueFactory.java Tue Sep 16 23:11:40 2014 +0300 2.2 +++ b/generaldb/src/main/java/org/openrdf/sail/generaldb/GeneralDBValueFactory.java Tue Sep 16 23:14:57 2014 +0300 2.3 @@ -372,13 +372,13 @@ 2.4 * FIXME my addition 2.5 * @throws ParseException 2.6 */ 2.7 - 2.8 public GeneralDBPolyhedron getRdbmsPolyhedron(Number num, String datatype, byte[] wkb, int srid) { 2.9 2.10 Number id = ids.idOf(num); 2.11 try { 2.12 - if(wkb != null) 2.13 + if(wkb != null) { 2.14 return new GeneralDBPolyhedron(id, literals.getIdVersion(), vf.createURI(datatype), wkb, srid); 2.15 + } 2.16 } catch (IOException e) { 2.17 e.printStackTrace(); 2.18 } catch (ClassNotFoundException e) { 2.19 @@ -386,20 +386,4 @@ 2.20 } 2.21 return null; 2.22 } 2.23 - 2.24 -// public GeneralDBPolyhedron getRdbmsPolyhedron(Number num, String datatype, String pointSet) { 2.25 -// 2.26 -// Number id = ids.idOf(num); 2.27 -// try { 2.28 -// return new GeneralDBPolyhedron(id, literals.getIdVersion(), vf.createURI(datatype), pointSet); 2.29 -// } catch (IOException e) { 2.30 -// e.printStackTrace(); 2.31 -// } catch (ClassNotFoundException e) { 2.32 -// e.printStackTrace(); 2.33 -// } 2.34 -// return null; 2.35 -// } 2.36 - /** 2.37 - * 2.38 - */ 2.39 }
3.1 --- a/monetdb/src/main/java/org/openrdf/sail/monetdb/evaluation/MonetDBQueryBuilder.java Tue Sep 16 23:11:40 2014 +0300 3.2 +++ b/monetdb/src/main/java/org/openrdf/sail/monetdb/evaluation/MonetDBQueryBuilder.java Tue Sep 16 23:14:57 2014 +0300 3.3 @@ -397,7 +397,7 @@ 3.4 3.5 //SRID support must be explicitly added! 3.6 return this; 3.7 - } 3.8 + } 3.9 3.10 3.11 @Override
4.1 --- a/postgis/src/main/java/org/openrdf/sail/postgis/evaluation/PostGISQueryBuilder.java Tue Sep 16 23:11:40 2014 +0300 4.2 +++ b/postgis/src/main/java/org/openrdf/sail/postgis/evaluation/PostGISQueryBuilder.java Tue Sep 16 23:14:57 2014 +0300 4.3 @@ -130,6 +130,7 @@ 4.4 public static final String ST_ASBINARY = "ST_AsBinary"; 4.5 public static final String GEOGRAPHY = "Geography"; 4.6 public static final String GEOMETRY = "Geometry"; 4.7 + 4.8 /** 4.9 * If (spatial) label column met is null, I must not try to retrieve its srid. 4.10 * Opting to ask for 'null' instead 4.11 @@ -827,21 +828,20 @@ 4.12 appendGeneralDBSpatialFunctionUnary(expr, filter, SpatialFunctionsPostGIS.ST_AsGML); 4.13 } 4.14 4.15 - // @Override 4.16 - // protected void append(GeneralDBSqlGeoSrid expr, GeneralDBSqlExprBuilder filter) 4.17 - // throws UnsupportedRdbmsOperatorException 4.18 - // { 4.19 - // appendGeneralDBSpatialFunctionUnary(expr, filter, SpatialFunctionsPostGIS.ST_SRID); 4.20 - // } 4.21 - 4.22 +/* 4.23 + @Override 4.24 + protected void append(GeneralDBSqlGeoSrid expr, GeneralDBSqlExprBuilder filter) 4.25 + throws UnsupportedRdbmsOperatorException 4.26 + { 4.27 + appendGeneralDBSpatialFunctionUnary(expr, filter, SpatialFunctionsPostGIS.ST_SRID); 4.28 + } 4.29 +*/ 4.30 /** 4.31 - * Special Case because I need to retrieve a single different column from geo_values when this function occurs 4.32 - * in the select clause 4.33 + * Special case because I need to retrieve a single different column from geo_values when this function occurs 4.34 + * in the select clause and not call the st_srid() function, which will always give me 4326. 4.35 */ 4.36 @Override 4.37 - protected void append(GeneralDBSqlGeoSrid expr, GeneralDBSqlExprBuilder filter) 4.38 - throws UnsupportedRdbmsOperatorException 4.39 - { 4.40 + protected void append(GeneralDBSqlGeoSrid expr, GeneralDBSqlExprBuilder filter) throws UnsupportedRdbmsOperatorException { 4.41 boolean sridNeeded = true; 4.42 filter.openBracket(); 4.43 4.44 @@ -905,6 +905,7 @@ 4.45 alias=alias+".srid"; 4.46 } 4.47 sridExpr = alias; 4.48 + 4.49 filter.append(sridExpr); 4.50 filter.closeBracket(); 4.51 return; 4.52 @@ -960,7 +961,7 @@ 4.53 } 4.54 4.55 filter.closeBracket(); 4.56 - } 4.57 + } 4.58 4.59 @Override 4.60 protected void append(GeneralDBSqlGeoIsSimple expr, GeneralDBSqlExprBuilder filter) 4.61 @@ -2197,7 +2198,7 @@ 4.62 filter.openBracket(); 4.63 if(expr.getArg() instanceof GeneralDBStringValue) 4.64 { 4.65 - appendWKT(expr.getArg(),filter); 4.66 + appendWKT(expr.getArg(), filter); 4.67 } 4.68 else if(expr.getArg() instanceof GeneralDBSqlSpatialConstructBinary) 4.69 { 4.70 @@ -2214,7 +2215,7 @@ 4.71 else if(expr.getArg() instanceof GeneralDBSqlCase) 4.72 { 4.73 GeneralDBLabelColumn onlyLabel = (GeneralDBLabelColumn)((GeneralDBSqlCase)expr.getArg()).getEntries().get(0).getResult(); 4.74 - appendMBB(onlyLabel,filter); 4.75 + appendMBB(onlyLabel, filter); 4.76 } 4.77 else 4.78 {