[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2018-03-09 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16393896#comment-16393896 ] Hudson commented on PHOENIX-4148: - SUCCESS: Integrated in Jenkins build PreCommit-PHOENIX-Build #1797

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2018-03-09 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16393884#comment-16393884 ] Hudson commented on PHOENIX-4148: - FAILURE: Integrated in Jenkins build Phoenix-4.x-HBase-1.3 #55 (See

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2018-03-07 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16390289#comment-16390289 ] James Taylor commented on PHOENIX-4148: --- Attached final patch with test and specific error code

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2018-02-22 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16373707#comment-16373707 ] James Taylor commented on PHOENIX-4148: --- I'll go ahead and commit your patch unless I hear

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-12-23 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16302570#comment-16302570 ] James Taylor commented on PHOENIX-4148: --- Just commit what you have - lgtm. > COUNT(DISTINCT(...))

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-12-23 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16302537#comment-16302537 ] Lars Hofhansl commented on PHOENIX-4148: Dropped the ball on this. I think it's important to fix

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-10-20 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16213321#comment-16213321 ] James Taylor commented on PHOENIX-4148: --- [~lhofhansl] - I think the patch is fine. I'm fine if you

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-09-13 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16164979#comment-16164979 ] Lars Hofhansl commented on PHOENIX-4148: Hey [~jamestaylor], this is the issue I mentioned. I do

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-09-11 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16161698#comment-16161698 ] Lars Hofhansl commented on PHOENIX-4148: I have a clumsy patch. (one can check for the size of

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-09-01 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16151366#comment-16151366 ] Lars Hofhansl commented on PHOENIX-4148: Hmm... How do I find the tenant id all the way down in

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-09-01 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16151241#comment-16151241 ] James Taylor commented on PHOENIX-4148: --- Yes, that sounds right. The MemoryManager is really just

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-09-01 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16151238#comment-16151238 ] Lars Hofhansl commented on PHOENIX-4148: My idea was to reserve a chunk of the HEAP (maybe in

[jira] [Commented] (PHOENIX-4148) COUNT(DISTINCT(...)) should have a memory size limit

2017-09-01 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16150119#comment-16150119 ] James Taylor commented on PHOENIX-4148: --- Use null as the tenantId. See example in