[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15932048#comment-15932048
]
Chia-Ping Tsai commented on HBASE-15525:
see HBASE-17805
> OutOfMemory could occur when using
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15931986#comment-15931986
]
stack commented on HBASE-15525:
---
[~anoop.hbase] [~chia7712] New issue?
> OutOfMemory could occur when
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15931856#comment-15931856
]
Anoop Sam John commented on HBASE-15525:
We use ByteBufferPool now which is serving fixed sized
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15931696#comment-15931696
]
Chia-Ping Tsai commented on HBASE-15525:
Should we remove the BoundedByteBufferPool ? It is
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15329506#comment-15329506
]
Anoop Sam John commented on HBASE-15525:
{quote}
// TODO better config names?
+ //
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15329366#comment-15329366
]
Hadoop QA commented on HBASE-15525:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15328996#comment-15328996
]
Anoop Sam John commented on HBASE-15525:
Thanks Stack.. Ya I checked again. Seems we call
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15327849#comment-15327849
]
stack commented on HBASE-15525:
---
Ok. That makes sense.
Isn't the close in closeAndPutbackBuffers
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15327146#comment-15327146
]
Hadoop QA commented on HBASE-15525:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15326944#comment-15326944
]
Anoop Sam John commented on HBASE-15525:
If the reasoning sounds ok, I will attach latest patch
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15326934#comment-15326934
]
Anoop Sam John commented on HBASE-15525:
There is reason why not close() but diff method is doing
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15326880#comment-15326880
]
Anoop Sam John commented on HBASE-15525:
Thanks [~dvdreddy] for the tests and update.
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15326858#comment-15326858
]
stack commented on HBASE-15525:
---
[~anoop.hbase] That is pretty nice endorsement of your patch given above.
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15326841#comment-15326841
]
deepankar commented on HBASE-15525:
---
Pulled in the latest patch and deployed on one machine, working as
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15309997#comment-15309997
]
Hadoop QA commented on HBASE-15525:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15296862#comment-15296862
]
Anoop Sam John commented on HBASE-15525:
I wanted to get it logged in INFO level at least once.
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15277723#comment-15277723
]
ramkrishna.s.vasudevan commented on HBASE-15525:
bq.Not a big deal and wont consume much
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15276508#comment-15276508
]
Anoop Sam John commented on HBASE-15525:
Ya we can consider DBB as come from pool ideally.. But
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15276495#comment-15276495
]
ramkrishna.s.vasudevan commented on HBASE-15525:
Great work!! Looks great.
Few questions
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15274380#comment-15274380
]
Ted Yu commented on HBASE-15525:
lgtm
{code}
87 if (LOG.isInfoEnabled()) {
88
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15274057#comment-15274057
]
Hadoop QA commented on HBASE-15525:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15273688#comment-15273688
]
deepankar commented on HBASE-15525:
---
We pulled in this patch and ran it on one of our production
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15261757#comment-15261757
]
Anoop Sam John commented on HBASE-15525:
bq.implementation than the one I created (and you
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15261677#comment-15261677
]
Hiroshi Ikeda commented on HBASE-15525:
---
I expect you create better implementation than the one I
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15234536#comment-15234536
]
Anoop Sam John commented on HBASE-15525:
Ya actually speaking, we might not need create a full
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15234529#comment-15234529
]
Anoop Sam John commented on HBASE-15525:
Clean way is to return the BB to pool in same place
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15234373#comment-15234373
]
Hiroshi Ikeda commented on HBASE-15525:
---
It is also better to pool heap buffers with weak
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15231863#comment-15231863
]
Hiroshi Ikeda commented on HBASE-15525:
---
It is better to keep the pool in BufferChain or something
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15228825#comment-15228825
]
Anoop Sam John commented on HBASE-15525:
I see in test result
bq.Tests run: 102, Failures: 0,
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15228167#comment-15228167
]
Hadoop QA commented on HBASE-15525:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15226923#comment-15226923
]
Hadoop QA commented on HBASE-15525:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15218608#comment-15218608
]
deepankar commented on HBASE-15525:
---
Sure happy to help
> OutOfMemory could occur when using
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15218459#comment-15218459
]
Anoop Sam John commented on HBASE-15525:
Ya we overflow on to heap area.. Ya I need to add some
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15218139#comment-15218139
]
stack commented on HBASE-15525:
---
bq. This is some thing I discussed with you offline some time back
Sorry
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15217435#comment-15217435
]
deepankar commented on HBASE-15525:
---
Oh ok, sorry for the confusion thanks
> OutOfMemory could occur
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15217425#comment-15217425
]
Anoop Sam John commented on HBASE-15525:
No I mean.. You pointed out issue when a Cell size is >
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15217417#comment-15217417
]
deepankar commented on HBASE-15525:
---
bq. Sure I will correct that.. One issue is that when we have
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15217392#comment-15217392
]
Anoop Sam John commented on HBASE-15525:
Sure I will correct that.. One issue is that when we
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15217378#comment-15217378
]
deepankar commented on HBASE-15525:
---
A minor comment on the patch for the
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15215397#comment-15215397
]
Anoop Sam John commented on HBASE-15525:
Yes Stack, while reading request into the BB and
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15215009#comment-15215009
]
stack commented on HBASE-15525:
---
So, just to note, on read side, we are already reading in 'chunks'. If you
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15214538#comment-15214538
]
stack commented on HBASE-15525:
---
Oh, so, it addresses this issue because sizing of the offheap pool is
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15214526#comment-15214526
]
stack commented on HBASE-15525:
---
So in your patch, ByteBufferPool is a FixedSizeBBP?
So we allocate in a
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15209785#comment-15209785
]
Anoop Sam John commented on HBASE-15525:
For writing the CellBlock response, we really dont need
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15209775#comment-15209775
]
Anoop Sam John commented on HBASE-15525:
The issue why we go out of off heap memory is because of
[
https://issues.apache.org/jira/browse/HBASE-15525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15209753#comment-15209753
]
Anoop Sam John commented on HBASE-15525:
I made it critical because it make it very difficult for
46 matches
Mail list logo