From 3ef4ddd5a04b2e045c89842f9e0adf43203b842d Mon Sep 17 00:00:00 2001
From: Greg Burd <gburd@onshape.com>
Date: Thu, 12 Oct 2017 09:23:00 -0400
Subject: [PATCH] Selections from materialized views my have different entity
 properties that impact generated queries.  Remove noisy/expensive selection
 of ttl and write time until we need/use it for cache correctness properties
 (if ever).

---
 .../helenus/core/operation/SelectOperation.java    | 14 ++++++++------
 .../core/views/MaterializedViewTest.java           |  2 +-
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/src/main/java/net/helenus/core/operation/SelectOperation.java b/src/main/java/net/helenus/core/operation/SelectOperation.java
index f36dc60..5444262 100644
--- a/src/main/java/net/helenus/core/operation/SelectOperation.java
+++ b/src/main/java/net/helenus/core/operation/SelectOperation.java
@@ -133,12 +133,12 @@ public final class SelectOperation<E> extends AbstractFilterStreamOperation<E, S
     Objects.requireNonNull(materializedViewClass);
     HelenusEntity entity = Helenus.entity(materializedViewClass);
     this.alternateTableName = entity.getName().toCql();
-    this.allowFiltering = true;
-    return this;
-  }
-
-  public SelectOperation<E> from(String alternateTableName) {
-    this.alternateTableName = alternateTableName;
+    this.props.clear();
+    entity
+        .getOrderedProperties()
+        .stream()
+        .map(p -> new HelenusPropertyNode(p, Optional.empty()))
+        .forEach(p -> this.props.add(p));
     return this;
   }
 
@@ -243,6 +243,7 @@ public final class SelectOperation<E> extends AbstractFilterStreamOperation<E, S
                 + prop.getEntity().getMappingInterface());
       }
 
+      /* TODO: is this useful information to gather when caching?
       if (cached) {
         switch (prop.getProperty().getColumnType()) {
           case PARTITION_KEY:
@@ -261,6 +262,7 @@ public final class SelectOperation<E> extends AbstractFilterStreamOperation<E, S
             break;
         }
       }
+      */
     }
 
     if (entity == null) {
diff --git a/src/test/java/net/helenus/test/integration/core/views/MaterializedViewTest.java b/src/test/java/net/helenus/test/integration/core/views/MaterializedViewTest.java
index 74f02fa..41c8eff 100644
--- a/src/test/java/net/helenus/test/integration/core/views/MaterializedViewTest.java
+++ b/src/test/java/net/helenus/test/integration/core/views/MaterializedViewTest.java
@@ -70,7 +70,7 @@ public class MaterializedViewTest extends AbstractEmbeddedCassandraTest {
   public void testMv() throws Exception {
     session
         .select(Cyclist.class)
-        .<CyclistsByAge>from(CyclistsByAge.class)
+        .from(CyclistsByAge.class)
         .where(cyclist::age, eq(18))
         .sync();
   }