diff --git a/querydsl-core/src/main/java/com/mysema/query/types/expr/Expr.java b/querydsl-core/src/main/java/com/mysema/query/types/expr/Expr.java
index ca15f5526..dd5a7b8bc 100644
--- a/querydsl-core/src/main/java/com/mysema/query/types/expr/Expr.java
+++ b/querydsl-core/src/main/java/com/mysema/query/types/expr/Expr.java
@@ -38,9 +38,7 @@ public abstract class Expr<D> implements Serializable{
      */
     @SuppressWarnings("unchecked")
     public static <T> Expr<T> create(T val){
-        if (val instanceof String){
-            return (Expr<T>)EString.create((String)val);
-        }else if (val instanceof Boolean){
+        if (val instanceof Boolean){
             return (Expr<T>)EBoolean.create((Boolean)val);            
         }else{
             return new ExprConst<T>(Assert.notNull(val,"val is null"));    
diff --git a/querydsl-core/src/main/java/com/mysema/query/types/path/PathMetadata.java b/querydsl-core/src/main/java/com/mysema/query/types/path/PathMetadata.java
index 6b684479e..2366c248f 100644
--- a/querydsl-core/src/main/java/com/mysema/query/types/path/PathMetadata.java
+++ b/querydsl-core/src/main/java/com/mysema/query/types/path/PathMetadata.java
@@ -17,9 +17,7 @@ import org.apache.commons.lang.builder.HashCodeBuilder;
 
 import com.mysema.commons.lang.Assert;
 import com.mysema.query.types.expr.ENumber;
-import com.mysema.query.types.expr.EString;
 import com.mysema.query.types.expr.Expr;
-import com.mysema.query.types.expr.ExprConst;
 import com.mysema.query.util.NotEmpty;
 
 /**
@@ -54,15 +52,17 @@ public final class PathMetadata<T> implements Serializable{
     }
 
     public static <KT> PathMetadata<KT> forMapAccess(PMap<?, ?> parent, KT key) {
-        return new PathMetadata<KT>(parent,ExprConst.create(key), PathType.MAPVALUE_CONSTANT);
+        return new PathMetadata<KT>(parent, Expr.create(key), PathType.MAPVALUE_CONSTANT);
     }
 
     public static PathMetadata<String> forProperty(Path<?> parent, @NotEmpty String property) {
-        return new PathMetadata<String>(parent, EString.create(Assert.hasLength(property)), PathType.PROPERTY);
+        // TODO : cache expressions ?
+        return new PathMetadata<String>(parent, Expr.create(Assert.hasLength(property)), PathType.PROPERTY);
     }
 
     public static PathMetadata<String> forVariable(@NotEmpty String variable) {
-        return new PathMetadata<String>(null, EString.create(Assert.hasLength(variable)), PathType.VARIABLE);
+        // TODO : cache expressions ?
+        return new PathMetadata<String>(null, Expr.create(Assert.hasLength(variable)), PathType.VARIABLE);
     }
 
     private final Expr<T> expression;