Comments on: HBase to the rescue http://airodig.com/2009/08/10/hbase-to-the-rescue/ airodig hacking industries Tue, 31 Aug 2010 20:17:16 +0000 hourly 1 http://wordpress.org/?v=3.0.1 By: Matt http://airodig.com/2009/08/10/hbase-to-the-rescue/comment-page-1/#comment-268 Matt Fri, 26 Mar 2010 22:16:18 +0000 http://airodig.com/?p=56#comment-268 >> I could straight shoot someone if I saw one more getter/setter pair. If you think I’m full of shit please read this (<a href="http://www.pragprog.com/articles/tell-dont-ask" rel="nofollow">http://www.pragprog.com/articles/tell-dont-ask</a>)<br><br>That article speaks specifically about retaining encapsulation, and is COMPLETELY THE OPPOSITE of what you were trying to state. >> I could straight shoot someone if I saw one more getter/setter pair. If you think I’m full of shit please read this (http://www.pragprog.com/articles/tell-dont-ask)

That article speaks specifically about retaining encapsulation, and is COMPLETELY THE OPPOSITE of what you were trying to state.

]]> By: Bradford http://airodig.com/2009/08/10/hbase-to-the-rescue/comment-page-1/#comment-20 Bradford Tue, 11 Aug 2009 05:59:50 +0000 http://airodig.com/?p=56#comment-20 This article is full of win and awesome. Check out my blog for HBasey goodness as well :D http://www.roadtofailure.com This article is full of win and awesome. Check out my blog for HBasey goodness as well :D

http://www.roadtofailure.com

]]> By: feydr http://airodig.com/2009/08/10/hbase-to-the-rescue/comment-page-1/#comment-19 feydr Tue, 11 Aug 2009 01:49:01 +0000 http://airodig.com/?p=56#comment-19 Thanks for the comments Ryan! Last Friday I was pretty dismayed at not having the ability to scan on start and stop timestamps so I patched the generated code there. I have no clue why I thought that someone wrote it considering it DOES say 'generated' -- my apologies for any offense taken. Anyways, I do have patches for the generated code at http://github.com/feydr/hrb/tree/3e17e6405ea6cd195319ae39d30247addd21751b/hrb-ng/patches . It was relatively easy to patch since it was already calling setTimeRange (like the java api does) -- it was just defaulting the start timestamp to a LONG.MIN_VALUE. Maybe I should have went through and found the original non-generated source ---- ughhh!! -- Perhaps I'll do that tmo. Thanks for the comments Ryan!

Last Friday I was pretty dismayed at not having the ability to scan on start and stop timestamps so I patched the generated code there. I have no clue why I thought that someone wrote it considering it DOES say ‘generated’ — my apologies for any offense taken. Anyways, I do have patches for the generated code at http://github.com/feydr/hrb/tree/3e17e6405ea6cd195319ae39d30247addd21751b/hrb-ng/patches . It was relatively easy to patch since it was already calling setTimeRange (like the java api does) — it was just defaulting the start timestamp to a LONG.MIN_VALUE. Maybe I should have went through and found the original non-generated source —- ughhh!! — Perhaps I’ll do that tmo.

]]> By: ryan http://airodig.com/2009/08/10/hbase-to-the-rescue/comment-page-1/#comment-18 ryan Tue, 11 Aug 2009 00:46:03 +0000 http://airodig.com/?p=56#comment-18 hey, if you have thrift patches, i'd be happy to review them. we did a number to the thrift api in 0.20, and while we tried to ensure all use cases in 0.20 worked via thrift, it appears there were some missed points. Rewriting the entire read/write path and nearly the entire regionserver makes for good bug fodder. Another thing to consider, is that the entire thrift generated/ subdir is generated code, so don't blame us :-) hey, if you have thrift patches, i’d be happy to review them. we did a number to the thrift api in 0.20, and while we tried to ensure all use cases in 0.20 worked via thrift, it appears there were some missed points. Rewriting the entire read/write path and nearly the entire regionserver makes for good bug fodder.

Another thing to consider, is that the entire thrift generated/ subdir is generated code, so don’t blame us :-)

]]> By: Twitted by feydr http://airodig.com/2009/08/10/hbase-to-the-rescue/comment-page-1/#comment-17 Twitted by feydr Mon, 10 Aug 2009 18:55:25 +0000 http://airodig.com/?p=56#comment-17 [...] This post was Twitted by feydr [...] [...] This post was Twitted by feydr [...]

]]>