[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

[GitHub] phoenix pull request: PHOENIX-2256: Correct two bugs in PMetaDataI...

2015-10-06 Thread chiastic-security
Github user chiastic-security closed the pull request at: https://github.com/apache/phoenix/pull/118 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] phoenix pull request: PHOENIX-2256: Correct two bugs in PMetaDataI...

2015-10-06 Thread chiastic-security
Github user chiastic-security commented on the pull request: https://github.com/apache/phoenix/pull/118#issuecomment-145763579 Thanks. Did this get pushed? The PR is still coming up as open in github. --- If your project is set up for it, you can reply to this email and have your

[GitHub] phoenix pull request: PHOENIX-2256: Correct two bugs in PMetaDataI...

2015-10-05 Thread chiastic-security
Github user chiastic-security commented on a diff in the pull request: https://github.com/apache/phoenix/pull/118#discussion_r41122346 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/PMetaDataImpl.java --- @@ -160,7 +160,7 @@ public PMetaDataCache cloneMinusOverage

[GitHub] phoenix pull request: PHOENIX-2256: Correct two bugs in PMetaDataI...

2015-10-02 Thread chiastic-security
Github user chiastic-security commented on a diff in the pull request: https://github.com/apache/phoenix/pull/118#discussion_r41041802 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/PMetaDataImpl.java --- @@ -160,7 +160,7 @@ public PMetaDataCache cloneMinusOverage

[GitHub] phoenix pull request: PHOENIX-2256: Correct two bugs in PMetaDataI...

2015-10-02 Thread chiastic-security
Github user chiastic-security commented on a diff in the pull request: https://github.com/apache/phoenix/pull/118#discussion_r41040085 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/PMetaDataImpl.java --- @@ -160,7 +160,7 @@ public PMetaDataCache cloneMinusOverage

[GitHub] phoenix pull request: Correct two bugs in PMetaDataImpl

2015-10-02 Thread chiastic-security
GitHub user chiastic-security opened a pull request: https://github.com/apache/phoenix/pull/118 Correct two bugs in PMetaDataImpl More entries than necessary could end up being evicted. This PR also adds a new test to catch one of the bugs. Previously, one of the