[GitHub] phoenix pull request: PHOENIX-2257 Fix broken integration test for...

2015-10-23 Thread maryannxue
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] phoenix pull request: PHOENIX-2257 Fix broken integration test for...

2015-10-23 Thread maryannxue
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] phoenix pull request: PHOENIX-2257 Fix broken integration test for...

2015-10-06 Thread JamesRTaylor
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] phoenix pull request: PHOENIX-2257 Fix broken integration test for...

2015-10-06 Thread JamesRTaylor
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] phoenix pull request: PHOENIX-2257 Fix broken integration test for...

2015-10-06 Thread JamesRTaylor
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] phoenix pull request: PHOENIX-2257 Fix broken integration test for...

2015-10-06 Thread chiastic-security
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