Github user maryannxue commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/120#discussion_r42860245
--- Diff:
phoenix-core/src/it/java/org/apache/phoenix/end2end/DerivedTableIT.java ---
@@ -313,21 +290,25 @@ public void testDerivedTableWithGroupBy() thro
Github user maryannxue commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/120#discussion_r42859436
--- Diff:
phoenix-core/src/it/java/org/apache/phoenix/end2end/DerivedTableIT.java ---
@@ -313,21 +290,25 @@ public void testDerivedTableWithGroupBy() thro
Github user JamesRTaylor commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/120#discussion_r41285442
--- Diff:
phoenix-core/src/it/java/org/apache/phoenix/end2end/DerivedTableIT.java ---
@@ -313,21 +290,25 @@ public void testDerivedTableWithGroupBy() th
Github user JamesRTaylor commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/120#discussion_r41285998
--- Diff:
phoenix-core/src/it/java/org/apache/phoenix/end2end/DerivedTableIT.java ---
@@ -313,21 +290,25 @@ public void testDerivedTableWithGroupBy() th
Github user JamesRTaylor commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/120#discussion_r41285023
--- Diff:
phoenix-core/src/it/java/org/apache/phoenix/end2end/DerivedTableIT.java ---
@@ -54,14 +28,17 @@
import org.junit.runners.Parameterized;
GitHub user chiastic-security opened a pull request:
https://github.com/apache/phoenix/pull/120
PHOENIX-2257 Fix broken integration test for query over derived GROUPâ¦
⦠BY table (testDerivedTableWithGroupBy()), which worked on Java 7 but
not on Java 8.
The test was retr