[DISCUSS] Thrift 0.9.1 in Accumulo 1.6.1

2014-05-16 Thread Christopher
Devs, I'm considering whether or not it'd be appropriate to push in ACCUMULO-1691 into the 1.6.1-SNAPSHOT branch. This would effectively bump our dependency on libthrift to 0.9.1. However, thrift 0.9.1 and 0.9.0 are 100% wire-compatible (I've been assured by jfarrell and codesf in the #thrift IRC

Re: Delays in email delivery?

2014-05-16 Thread Sean Mackrory
I've been seeing delays, and another user on the Bigtop mailing list reported similar problems a short time ago. I also got a notice yesterday that emails from the mailing list were bouncing back, and emails from several other committers with whom I communicate regularly are getting flagged as poss

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Christopher
That looks like an accurate representation of the informal process. +1 for letting them assign themselves Aside about #7: If this process gets documented in a contributor section of the website (which, maybe it should?) or a wiki, or whatever, I'd want to just add note that when looking for ticket

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Josh Elser
> On May 16, 2014, 8:19 p.m., Josh Elser wrote: > > server/tserver/src/test/java/org/apache/accumulo/tserver/log/LocalWALRecoveryTest.java, > > line 75 > > > > > > Wouldn't it have simplified things to just use the Raw

Re: SQL layer over Accumulo?

2014-05-16 Thread James Taylor
That sounds promising, Josh & William. Is there a performance penalty with this approach (versus traversing the rows in row key order)? Thanks, James On Fri, May 16, 2014 at 8:27 AM, Josh Elser wrote: > On 5/11/14, 12:22 AM, James Taylor wrote: > >> @William - it's entirely possible that my HBa

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Josh Elser
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21567/#review43298 --- Test now passes for me - Josh Elser On May 16, 2014, 11:29 p.m.,

Re: Review Request 21169: ACCUMULO-2770 Add utility to read local WAL

2014-05-16 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21169/ --- (Updated May 7, 2014, 11:33 p.m.) Review request for accumulo, Sean Busbey and

Re: Review Request 21557: ACCUMULO-2766 fix wal group commit

2014-05-16 Thread keith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21557/ --- (Updated May 16, 2014, 10:56 p.m.) Review request for accumulo. Changes -

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Sean Busbey
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21567/ --- (Updated May 16, 2014, 11:29 p.m.) Review request for accumulo, Josh Elser and

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Sean Busbey
> On May 16, 2014, 8:19 p.m., Josh Elser wrote: > > server/tserver/src/test/java/org/apache/accumulo/tserver/log/LocalWALRecoveryTest.java, > > line 75 > > > > > > Wouldn't it have simplified things to just use the Raw

Re: Review Request 21282: ACCUMULO-2791 Downgrade commons-codec to match that provided by Hadoop.

2014-05-16 Thread Sean Busbey
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21282/ --- (Updated May 9, 2014, 11:39 p.m.) Review request for accumulo. Changes --

Re: Tablet Redistribution

2014-05-16 Thread Sean Busbey
the default balancer should already handle moving the tablets across all servers. Check the master's debug log to see if there are messages about trouble with balancing. For example, if there are unhosted tablets then the master won't rebalance tablets. If you have one or more offline tables, you

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Sean Busbey
> On May 16, 2014, 9:13 p.m., Mike Drob wrote: > > server/tserver/src/main/java/org/apache/accumulo/tserver/logger/LogFileKey.java, > > lines 44-46 > > > > > > I'd really like to move this away from using ordinals, but

Re: Review Request 21282: ACCUMULO-2791 Downgrade commons-codec to match that provided by Hadoop.

2014-05-16 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21282/#review42605 --- core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/im

Re: Review Request 21557: ACCUMULO-2766 fix wal group commit

2014-05-16 Thread keith
> On May 16, 2014, 7:11 p.m., Mike Drob wrote: > > server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java, > > line 150 > > > > > > Should explicitly handle InvocationTargetException. Can be done

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Josh Elser
Until we have problems with it, I would prefer that contributors could continue to assign things as they see fit. Their contributor status implies that they are at least somewhat acclimated with our processes and that they have a positive opinion of the project. On 5/14/14, 4:38 PM, Sean Bus

Re: Delays in email delivery?

2014-05-16 Thread Joey Echeverria
Infra noted a 1.5M long email queue for sending to gmail, so I think it's the interaction between the two. -- Joey Echeverria Chief Architect Cloudera Government Solutions On Fri, May 16, 2014 at 4:06 PM, Sean Mackrory wrote: > I've been seeing delays, and another user on the Bigtop mailing lis

Re: Has anyone looked at Spring-XD and Accumulo integration?

2014-05-16 Thread Sean Busbey
Kite's already integrated into Spring XD, so I was hoping that by getting things together there we'd have an easier time wiht things like Spring XD On Fri, May 9, 2014 at 2:33 PM, David Medinets wrote: > https://github.com/spring-projects/spring-xd is the project's page. > -- Sean

Re: Accumulo Blog Posts

2014-05-16 Thread dlmarion
You have a blog account (medined). You will probably need to submit an INFRA ticket to get the password reset. - Original Message - From: "David Medinets" To: "accumulo-dev" Sent: Friday, May 9, 2014 10:39:43 AM Subject: Accumulo Blog Posts I think the community decided to move f

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Sean Busbey
> On May 16, 2014, 8:19 p.m., Josh Elser wrote: > > server/tserver/src/test/java/org/apache/accumulo/tserver/log/LocalWALRecoveryTest.java, > > line 75 > > > > > > Wouldn't it have simplified things to just use the Raw

Delays in email delivery?

2014-05-16 Thread Michael Wall
I had been having trouble tracking some of the threads on both the user and dev Accumulo lists. I realized this morning it was because emails were being delayed. For example, David Medinets posted "Accumulo Blog Posts" to the dev list 6 days ago. I got Billie's response on the 12th. Received: f

Re: Review Request 21557: ACCUMULO-2766 fix wal group commit

2014-05-16 Thread keith
> On May 16, 2014, 5:30 p.m., John Vines wrote: > > server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java, > > line 153 > > > > > > I think that before this loop the closeLock and/or closed shoul

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21567/#review43266 --- Ship it! server/tserver/src/main/java/org/apache/accumulo/tserver/

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Josh Elser
> On May 16, 2014, 8:19 p.m., Josh Elser wrote: > > server/tserver/src/test/java/org/apache/accumulo/tserver/log/LocalWALRecoveryTest.java, > > line 75 > > > > > > Wouldn't it have simplified things to just use the Raw

Re: [DISCUSS] Thrift 0.9.1 in Accumulo 1.6.1

2014-05-16 Thread Christopher
Yes. They are 100% forward/backwards compatible on the wire. However, it is my understanding that there were some minor additions (new API) to 0.9.1 which won't work in 0.9.0... but that won't affect us since we are not using those features (and wouldn't be adding anything that leverages those fea

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Sean Busbey
> On May 16, 2014, 9:13 p.m., Mike Drob wrote: > > server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java, > > line 302 > > > > > > Was this a copy/paste error? it looks like it was an error in h

Tablet Redistribution

2014-05-16 Thread Jeff N
Is there a way to write Java code to redistribute tablets across all tablet servers? I've come across the TabletBalancer class in 1.5 and I'm currently trying to utilize it to evenly redistribute all of the tablets because they are clustered onto one tablet server. Is there an easier way to do this

Any scala users on the list?

2014-05-16 Thread David Medinets
I'd like to look into the level of effort to allow the Overview server ( https://www.overviewproject.org/) to pull text directly from Accumulo. This tools seems like a natural fit with the TedgeText table in the D4M schema.

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread John Vines
Yes, restore the old behavior On Wed, May 14, 2014 at 4:38 PM, Sean Busbey wrote: > We don't have a formal onboarding process for drawing in new contributors, > but a recent ASF Infra change impacts what I've observed historically. > > Here's what I've seen historically, more or less: > > 1) So

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Sean Busbey
> On May 16, 2014, 8:19 p.m., Josh Elser wrote: > > server/tserver/src/test/java/org/apache/accumulo/tserver/log/LocalWALRecoveryTest.java, > > line 75 > > > > > > Wouldn't it have simplified things to just use the Raw

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Billie Rinaldi
Luckily, I believe we are not affected by the change because we are using the Hadoop Permissions scheme in JIRA and not the ASF defaults. For a first-time contributor, I don't think it makes much difference whether we assign the ticket or they do. But it seems better for existing contributors to

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread David Medinets
What was the reasoning behind the change? On Wed, May 14, 2014 at 4:38 PM, Sean Busbey wrote: > We don't have a formal onboarding process for drawing in new contributors, > but a recent ASF Infra change impacts what I've observed historically. > > Here's what I've seen historically, more or les

Re: Review Request 21072: ACCUMULO-2635 - ZooCacheFactory (1.6.1)

2014-05-16 Thread keith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21072/#review43106 --- Ship it! Ship It! - kturner On May 5, 2014, 2:23 p.m., Bill Hava

Re: Review Request 21557: ACCUMULO-2766 fix wal group commit

2014-05-16 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21557/#review43239 --- server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLog

Re: Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Josh Elser
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21567/#review43255 --- server/tserver/src/test/java/org/apache/accumulo/tserver/log/LocalW

Review Request 21567: ACCUMULO-2816 Ensure LocalWALRecovery looks at WAL.

2014-05-16 Thread Sean Busbey
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21567/ --- Review request for accumulo, Josh Elser and Mike Drob. Bugs: ACCUMULO-2816

Re: Delays in email delivery?

2014-05-16 Thread Michael Wall
Nevermind, see https://blogs.apache.org/infra/entry/mail_outage I am not going crazy. On Thu, May 15, 2014 at 3:57 PM, Michael Wall wrote: > I had been having trouble tracking some of the threads on both the user > and dev Accumulo lists. I realized this morning it was because emails were > b

Re: [DISCUSS] Thrift 0.9.1 in Accumulo 1.6.1

2014-05-16 Thread Sean Busbey
Do we know if thirft 0.9.0 and 0.9.1 are forward compatible? On Fri, May 16, 2014 at 12:59 PM, Christopher wrote: > Correction: the current patch does *NOT* bump the wire version... I > thought I did that, but I did not. > > -- > Christopher L Tubbs II > http://gravatar.com/ctubbsii > > > On Fr

Review Request 21557: ACCUMULO-2766 fix wal group commit

2014-05-16 Thread keith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21557/ --- Review request for accumulo. Bugs: ACCUMULO-2766 https://issues.apache.org/

Re: Review Request 21557: ACCUMULO-2766 fix wal group commit

2014-05-16 Thread keith
> On May 16, 2014, 5:30 p.m., John Vines wrote: > > server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java, > > line 153 > > > > > > I think that before this loop the closeLock and/or closed shoul

Re: [DISCUSS] Thrift 0.9.1 in Accumulo 1.6.1

2014-05-16 Thread Christopher
Correction: the current patch does *NOT* bump the wire version... I thought I did that, but I did not. -- Christopher L Tubbs II http://gravatar.com/ctubbsii On Fri, May 16, 2014 at 12:28 PM, Christopher wrote: > Devs, > > I'm considering whether or not it'd be appropriate to push in > ACCUMULO

Re: Review Request 21557: ACCUMULO-2766 fix wal group commit

2014-05-16 Thread John Vines
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21557/#review43223 --- server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLog

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Corey Nolet
+1 for restoring old behavior.Why wouldn't we allow contributors to help themselves help the community? On Thu, May 15, 2014 at 11:13 AM, John Vines wrote: > Yes, restore the old behavior > > > On Wed, May 14, 2014 at 4:38 PM, Sean Busbey wrote: > > > We don't have a formal onboarding process

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Keith Turner
Looking at the ticket you linked it seems it easy for infra to make this change. As long as its not a hassle for infra I would be in favor of changing it back. The problem w/ issues being assigned to people who are not working on them (it seems this is sparks concern) applies to commiters and co

Re: SQL layer over Accumulo?

2014-05-16 Thread Josh Elser
On 5/11/14, 12:22 AM, James Taylor wrote: @William - it's entirely possible that my HBase terminology is not mapping well to Accumulo terminology. If Accumulo has a capability not present in HBase that'll handle this, that'd be great. In HBase terminology, by row I mean all of the key values acr

Re: [DISCUSS] DRAFT End of Life for Accumulo 1.4

2014-05-16 Thread Alex Moundalexis
The last paragraph, is it: a) an instruction to work on the 1.4 line? b) saying that work on the 1.4 line will cease? On Wed, May 7, 2014 at 11:50 AM, Sean Busbey wrote: > Presented for feedback / discussion: draft ANNOUNCE message for the > user@accumulo list. > > I think the Apache release rul

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Joey Echeverria
> It seems like a simple solution to that> is some documentation on our web > site encouraging people to contact > the person a ticket is assigned to if > they are interested in working on it.  +1 -- Joey Echeverria Chief Architect Cloudera Government Solutions On Fri, May 16, 2014 at 9:46

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Alex Moundalexis
As a contributor, I like this approach. I'm able to through and assign an issue to myself without pestering anyone else. I like the incentive-based approach for prospective contributors: have a committer assign an issue to a user after they express interest in working it, then give them contributo

Re: [DISCUSS] Do we want contributors assigning to themselves?

2014-05-16 Thread Joey Echeverria
I like that flow. I'm very much against not letting contributors assign tickets. If you have a problem with "old" contributors "stealing" tickets from new contributors, then that needs to be handled with best practices documentation and/or explicit coaching. Locking down who can assign tickets

Re: Review Request 21072: ACCUMULO-2635 - ZooCacheFactory (1.6.1)

2014-05-16 Thread keith
> On May 15, 2014, 1:53 p.m., kturner wrote: > > Ship It! I was concerned about wider sharing of zoocache objects than before AND calls to zoocahce.clear(). Looking at the code it seems that most of the code that calls zoocache.clear() creates its own zoocache via "new zoocache". So the cod

Re: [DISCUSS] Majority voting without prior discussion

2014-05-16 Thread Billie Rinaldi
I definitely agree with #1. Still pondering #2. On Tue, May 6, 2014 at 1:58 PM, Christopher wrote: > Devs, > > As something that came out of the vote thread about EOL'ing 1.4, I was > thinking: > > The purpose of a majority vote seems to be when we've already > discussed and planned, and we ju

Re: Review Request 21282: ACCUMULO-2791 Downgrade commons-codec to match that provided by Hadoop.

2014-05-16 Thread Sean Busbey
> On May 9, 2014, 9:22 p.m., Mike Drob wrote: > > core/src/test/java/org/apache/accumulo/core/client/mapreduce/AccumuloInputFormatTest.java, > > line 81 > > > > > > Specify UTF8. library always uses UTF8. - Sean -