Re: On ticket management

I agree wholeheartedly with John's thoughts on this. Having somebody act as the gatekeeper for each realm is useful, but it is not necessary to assign everything in that realm to the gatekeeper. It is my opinion that the "Assignee" field should refer to somebody who is either working on the issue

Re: Review Request: Quotes in visibility

er test that checks quotes + parenthesis? /trunk/core/src/test/java/org/apache/accumulo/core/security/VisibilityEvaluatorTest.java <https://reviews.apache.org/r/6861/#comment23455> Catch VisibilityParseException? - Mike Drob On Aug. 30, 2012, 8:59 p

Re: Double Checked Locking - broken? working?

It's a private static field that never changes once we do the initial load. The load looks idempotent, so I'm not sure it's worth the hassle of trying to synchronize on it, or even do it lazily. Maybe somebody with more history has seen that it is way more performance intensive than I give it credi

Re: Proper way to build an accumulo RPM?

Drew, When building the rpm itself, you want to run "mvn rpm:rpm -N" to specify no children. Otherwise, what you're doing looks very similar to commands I've had success with in the past. Mike On Fri, Oct 19, 2012 at 12:59 PM, Drew Farris wrote: > Hi All, > > Given the impending 1.4.2 release

Re: Setting Charset in getBytes() call.

One specific use case is when creating a new connection, the password is passed as a byte[], when I expect most sane applications will treat it as a String (either via reading it from a file, or reading it from a terminal input). If somebody creates the password with a different platform encoding t

Re: Accumulo 1.6 and beyond feature summit

Java 6 End of Public Updates is slated for Feb 2013 [1], so I'm sure there will be a lot of interest in being Java 7 compatible very soon. +1 for Java 7, bash commands, and Percolator [1]: http://www.oracle.com/technetwork/java/eol-135779.html On Tue, Jan 29, 2013 at 1:11 PM, John Vines wrote:

Re: ACCUMULO-958 - Pluggable encryption in walogs

I don't have any concerns with releasing features in 1.5.1, but please be cognizant that the intent of ACCUMULO-751was for wire compatibility across the 1.5.X series. I'll strongly agree with Keith regarding a stable API - confusing and constantl

Re: Is there a style file for NetBeans?

There's also the maven-formatter-plugin [1] which looks both awesome and incredibly powerful. [1]: https://code.google.com/p/maven-java-formatter-plugin/ On Tue, Mar 5, 2013 at 6:44 PM, Christopher wrote: > I was unable to find a way to convert our Eclipse formatter to a > checkstyle file... b

Re: GSOC: Monitor Improvements

Adding on to the comment about summaries, averages, and outliers. If, for some reason, you end up with a two-hump population, then simply showing averages will mask the split and lose a lot of valuable information. It is often valuable to know that a particular set of users or servers are experienc

JIRA Patch Conventions

Accumulo Devs, Are there any conventions that we'd like to follow for attaching updated patches to issues? There are two lines of thought applicable here: 1) Remove the old one and attach the new patch. This has the advantage of being immediately obvious to future google searchers what the patch

Re: Geographic Plot of Accumulo Downloads

Is it visits or visitors? On Fri, May 3, 2013 at 12:16 PM, Adam Fuchs wrote: > This covers about the last year: all time since Accumulo became a top-level > project. The metric is visits by location. > > Adam > > > > On Fri, May 3, 2013 at 10:46 AM, David Medinets >wrote: > > > Does that graph

Accumulo Versions in JIRA

Since we're doing 1.5.0 Release candidates, can somebody create a 1.5.1 version for JIRA?

Re: GIT

Looks like git support is available, but still a work in progress. See https://git-wip-us.apache.org/ On Tue, May 21, 2013 at 6:12 PM, Christopher wrote: > On Tue, May 21, 2013 at 6:05 PM, David Medinets > wrote: > > I thought we were constrained by apache's subversion repo? > > I'm not sure w

Re: svngit2jira for Accumulo?

Looks like it worked on ACCUMULO-1470. Looks so shiny and pretty! On Wed, May 29, 2013 at 10:19 AM, John Vines wrote: > Thanks Drew! > > > On Tue, May 28, 2013 at 9:56 PM, Drew Farris wrote: > > > It looks like everyone's happy with this. > > > > Ticket created: https://issues.apache.org/jira/

Re: [DISCUSS] The State of Old Versions

Nobody's touched this for a week, but I think the conversation is important enough that it shouldn't die. I'll tackle these in version order. 1.3 - I agree with releasing a source only tag of 1.3.7 and calling it end-of-life. The changes are there, throwing them away seems silly (and rude?). That

Re: [VOTE] JDK 1.7 - Switch for Accumulo 1.6.0

I think building the binary release using 1.7, but still maintaining 1.6 compatibility is a nice compromise. You're providing enough of a carrot to use 1.7 (hey look, we already built it for you), but you're not alienating the users that still need 1.6 classes for whatever reason. If something like

Re: [VOTE] JDK 1.7 - Switch for Accumulo 1.6.0

Also. -1 because I forgot that there was a vote open. On Wed, Jun 5, 2013 at 7:25 PM, Mike Drob wrote: > I think building the binary release using 1.7, but still maintaining 1.6 > compatibility is a nice compromise. You're providing enough of a carrot to > use 1.7 (hey look, we

Re: [git] Documentation and Plan of Action

How do we handle the following situation: There are two active development branches - versions 1 and 2. Branch 1 uses external library version 7, while Branch 2 uses external library version 9. It is discovered that version 7 of the external library has a bug, with a known workaround. In this cas

Re: [git] Documentation and Plan of Action

d in Version1 should ensure that > when Version1 is merged into Version2, the correct code is present in > Version2. > > Do you think this merits a section on the document or was this a general > question? > > > On 6/11/13 11:44 AM, Mike Drob wrote: > >> How do we handl

Re: [git] Documentation and Plan of Action

ing, after making the changes in version1, it is your > responsibility to make sure whatever necessary variants exist in their > appropriate versions. > > > On 06/11/2013 07:10 PM, Mike Drob wrote: > >> It was originally a general question, because I wasn't sure on what the >&

Re: Backporting policy proposal

I think the point about CTR is very important, and should be weighed carefully before making binding decisions regarding back-ports. If a developer wants to spend time back-porting features, then I don't want to be the one to dissuade them. Like I've heard on this list many times, "patches welcome!

Re: Accumulo build error

Do you have rpmbuild installed? On Jun 24, 2013 8:20 AM, "Andres Danter" wrote: > Hi Josh, > > That was very helpful. It got me passed the tests and my build was > successful. I'm now trying to build the RPM for Accumulo 1.4.3, but I'm > getting resistance there too. I'm running the command: >

Accumulo 1.4.4

I've created a JIRA for putting together a 1.4.4 release. https://issues.apache.org/jira/browse/ACCUMULO-1546 There are 9 outstanding issues at this time, some of which will likely slip into 1.4.5 if they are not addressed. If anybody has specific changes that they would like to see made, now is t

Re: New committer!

Thanks! I care a lot about fault tolerance and recovery, and creating easy operations tooling. I hope to continue to improve the shell and make interactive interactions with Accumulo easier. Mike On Wed, Jul 3, 2013 at 11:40 AM, Billie Rinaldi wrote: > I am pleased to announce that Mike D

Re: Accumulo 1.4.4

> > > On Mon, Jul 1, 2013 at 5:25 PM, Mike Drob wrote: > > > I've created a JIRA for putting together a 1.4.4 release. > > https://issues.apache.org/jira/browse/ACCUMULO-1546 > > > > There are 9 outstanding issues at this time, some of which will lik

Merging Practices

I tested and committed the patch for ACCUMULO-895 and am currently trying to update the svn mergeinfo into 1.5 and trunk. The bottom note tells me that I should ask the dev list for details and help merging, so here I am asking. I currently have conflicts for src/server and src/core and no idea how

Re: Merging Practices

doing a merge of "1:HEAD" might be too wide of a range. On Sat, Jul 6, 2013 at 6:05 PM, Josh Elser wrote: > I believe all you need to do by hand is to merge src/${module} in 1.4 to > ${module} in 1.5. You just can't merge 1.4 directly in 1.5. > > 1.5 to trunk should be s

Re: Merging Practices

EAD was clean, but since i don't remember when that > was, it was probably a while ago :P > > I don't think it would be out of line to just merge your changes. Probably > not a huge deal with the Git switchover (hopefully) imminent. > On Jul 6, 2013 6:57 PM, "Mike Drob&

Re: Accumulo 1.4.3 RPM

the RPM with the correct arch (change > 'true' to 'x86_64' in the > parent pom.xml). > > This change could be made for 1.4.4, I suppose, if it's not too late > to sneak in that fix (Mike Drob had mentioned trying to tag 1.4.4 > soon). If you can confirm this

Re: Accumulo 1.4.3 RPM

I > http://gravatar.com/ctubbsii > > > On Sun, Jul 7, 2013 at 9:03 PM, Mike Drob wrote: > > I've made this change independently, and I recall it working. I don't > know > > how this affects debian and other apt-based systems, however. > > > > > > On

Re: Schedule for 1.6.0 release?

I think we should shoot for a feature freeze two months before the release, so some time (probably the end of) September. That gets a good couple rounds of testing, bug fixes, and more testing in. On Thu, Jul 11, 2013 at 5:17 PM, Sean Busbey wrote: > One of the action items out of the 1.6.0 dis

Re: Thrift generated files

Technically, you can. However, this is considered very bad practice, as the next person to touch these is likely going to overwrite your changes and then we'll have a mess of conflicts to deal with. On Mon, Jul 22, 2013 at 11:14 PM, Supun Kamburugamuva wrote: > Just out of curiosity, without cop

Re: Local integration tests

Are you specifically asking for advice on how to run them more efficiently locally, or how to skip them so that you can run through the full build faster? On Tue, Jul 23, 2013 at 5:46 PM, Michael Berman wrote: > Does anyone have any advice for running integration tests locally? At the > moment

Re: Hadoop 2.0 Support for Accumulo 1.4 Branch

Which version of 0.20 are you testing against? Vanilla, or cdh3 flavored? On Fri, Aug 2, 2013 at 2:37 PM, Joey Echeverria wrote: > I don't think that's a good idea unless you can come up with very > clear version number change. > > -Joey > > On Fri, Aug 2, 2013 at 2:31 PM, Christopher wrote: >

Accumulo 1.4.4 RC1

Devs, I've made an initial attempt at building a 1.4.4-rc1 release. The src tar, signature, and md5sum are available at https://people.apache.org/~mdrob/ The tar was generated from the 1.4.4rc1 tag by running "git archive -o accumulo-1.4.4-rc1-src.tar.gz 1.4.4rc1" There is no vote yet because my

Re: Accumulo 1.4.4 RC1

T > > > > > > > > On 08/09/2013 09:33 PM, Josh Elser wrote: > > > >> Looks like bin/config.sh uses 1.4.4-SNAPSHOT and not 1.4.4 when setting > >> ACCUMULO_VERSION. > >> > >> On 08/07/2013 05:50 PM, Mike Drob wrote: > >>

[VOTE] Accumulo 1.4.4 RC3

solution was to just keep incrementing. -- Forwarded message -- From: Nexus Repository Manager Date: Thu, Aug 15, 2013 at 1:43 PM Subject: Nexus: Staging Completed. To: Mike Drob *Description:* Accumulo 1.4.4 RC3 *Details:* The following artifacts have been staged to the org

Re: [VOTE] Accumulo 1.4.4 RC3

e-only release. I would appreciate any other issues that we can identify with this RC before I start working on the next one. Mike On Thu, Aug 15, 2013 at 3:39 PM, Mike Drob wrote: > Devs, > > Please consider the following candidate for Accumulo 1.4.4. > > It was tagged using the ma

Re: [VOTE] Accumulo 1.4.4 RC3

No votes and a missing source tarball mean that this vote fails. On Fri, Aug 16, 2013 at 1:57 PM, Sean Busbey wrote: > On Fri, Aug 16, 2013 at 9:51 AM, Keith Turner wrote: > > > > > * master box is not showing up on monitor page. Maybe this is because > I > > am running Accumulo and hadoop

[VOTE] Accumulo 1.4.4 RC4

heaccumulo-107/ This vote will remain open for 72 hours. Mike -- Forwarded message -- From: Nexus Repository Manager Date: Tue, Aug 20, 2013 at 12:04 AM Subject: Nexus: Staging Completed. To: Mike Drob *Description:* accumulo 1.4.4 rc4 *Details:* The following artifacts have b

Re: [VOTE] Accumulo 1.4.4 RC4

wrote: > +1 > > Tag and staging repo both build as expected. > > Source and dist release look in order as well. > > Thanks for taking this on, Mike. > > > On 08/20/2013 12:28 PM, Mike Drob wrote: > >> Devs, >> >> Please consider the following ca

Re: [VOTE] Accumulo 1.4.4 RC4

? - Mike On Fri, Aug 23, 2013 at 3:45 PM, Mike Drob wrote: > VOTE passes with +4. (Myself, Eric, Christopher, Josh) > > Thank you guys for verifying. > > > The maven repo has been promoted to release. > > TODO: Add artifacts to the svn release repo; Move tag from 1.4.

Re: [VOTE] Accumulo 1.4.4 RC4

o address ACCUMULO-1532. > Alright, I'll leave the web manual and javadoc as is. For the PDF manual, do I just delete/upload the new version? > > Billie > > > On Sun, Aug 25, 2013 at 9:16 AM, Mike Drob wrote: > > > Update: > > > > The artifacts have

Re: SVN or GIT

+1 On Mon, Aug 26, 2013 at 3:41 PM, Keith Turner wrote: > On Mon, Aug 26, 2013 at 2:41 PM, Christopher wrote: > > > Since we're on git now... to avoid confusion like this in the future > > (not just from PMC/committers who may not have touched the code in > > awhile, but also from users lookin

Re: Schedule for 1.6.0 release?

I wanted to revive this conversation, since fall is fast approaching. One reasonable target for a release date might be to try and get something done before Hadoop World/Strata NY, which is the last week of October. That is a bit sooner than initially planned, but would be a great bit of PR if it w

Re: StatsIterator - what is the significance of 23?

At a guess, it looks like a small prime intended to decrease the amount of atomic operations. Completely unrelated, but this made me think of Feynman and the mysterious number 137. On Tue, Sep 3, 2013 at 9:29 PM, David Medinets wrote: > At the risk of having overlooked something obvious, I was

Re: Building trunk

Do you mean ACCUMULO-1672 ? On Mon, Sep 9, 2013 at 5:43 PM, John Vines wrote: > Anyone else having errors building trunk? Specifically, proxy tests failing > due to a constraint violation exception? >

Re: [VOTE] Accumulo 1.4.4 RC4

Did this ever get announced on the announce@a.o? Should it have been? On Sun, Aug 25, 2013 at 8:27 PM, Billie Rinaldi wrote: > On Aug 25, 2013 7:23 PM, "Mike Drob" wrote: > > > > Responses Inline. > > > > On Sun, Aug 25, 2013 at 2:03 PM, Billie Rinaldi

Accumulo Cluster Metrics

I know that Accumulo has capabilities for JMX and Ganglia integration, but are the metrics available documented somewhere? I checked the user manual but didn't see anything about them, and Google didn't help much either. -Mike

Re: RestartIT.restartMasterRecovery

While in general I agree with Sean that once resolved in a released version, JIRAs should stay resolved, I have no problem with re-opening issues that are marked resolved in a non-released version, since, well, they were never resolved. On Fri, Sep 13, 2013 at 6:20 PM, Sean Busbey wrote: > If t

Re: [VOTE] Accumulo Instamo Archetype 1.4.4

Josh, What role does this fill that the Accumulo maven plugin does not? They seem to address similar needs and I'm worried about duplicated effort. On Sep 17, 2013 11:00 PM, "Josh Elser" wrote: > Devs, > > Please consider the following candidate project as a "sister" release to > Apache Accumulo

Re: Schedule for 1.6.0 release?

> +1 > > -- > Christopher L Tubbs II > http://gravatar.com/ctubbsii > > > On Wed, Sep 18, 2013 at 10:36 AM, Keith Turner wrote: > > We do need to get this settled. What about end of year target for > release > > date and feature freeze date at end of Oct? > > > &g

Re: [VOTE] Accumulo Instamo Archetype 1.4.4

s in the accumulo repo which spin up a > minicluster instance in @BeforeClass and stop it in @AfterClass). > > Does that make sense? > > On 09/17/2013 11:10 PM, Mike Drob wrote: > >> Josh, >> >> What role does this fill that the Accumulo maven plugin does not? The

Re: [VOTE] Accumulo Instamo Archetype 1.4.4

-1 for no source tarball :) On Thu, Sep 19, 2013 at 11:33 AM, Josh Elser wrote: > In the spirit of doing this properly, I'm going to -1 myself, however > additional feedback is greatly appreciated to make a better final product. > > I'll wait the originally stated 72 hrs before putting up the n

Re: Schedule for 1.6.0 release?

on/glossary.html#MajorityApproval > > [2]:http://www.apache.org/foundation/glossary.html#ConsensusApproval > > > > - > > > > > > > >> > >> On Wed, Sep 18, 2013 at 2:14 PM, Christopher > wrote: > >> > >> > +1 > >

Re: [VOTE] Accumulo Instamo Archetype 1.4.4 RC2

#7 is exactly what I did for Accumulo 1.4.4 - I don't think the we should be tagging something as X is X has not been voted on. X-rcY (or some permutation) is fine. On Wed, Sep 25, 2013 at 3:45 PM, Christopher wrote: > #5, #6, and #7 need to be fixed. #1, #2, and #8 are trivial to fix > while a

Re: Would gerrit help to organize accumulo code reviews?

Was about to chime in with reviews.a.o, when I saw Sean beat me to it. It's mostly that and patches attached to tickets. Pull requests coming from github might also work, but I don't think anybody's tried them yet. On Fri, Sep 27, 2013 at 9:57 AM, Sean Busbey wrote: > Right now, if they're done

Re: [VOTE] 1.6.0 Feature freeze.

+1 On Fri, Sep 27, 2013 at 4:02 PM, Brian Loss wrote: > +1 > > On Sep 27, 2013, at 1:39 PM, John Vines > wrote: > > > Please vote on a feature freeze date of Nov 1 23:59 PDT for the master > > branch. Shortly after this time we will branch 1.6.0-SNAPSHOT from > master > > and increment the v

Re: integration tests

Maven already provides a -T option for multithreaded runs, and can even take a core multiplied factor. It would be nice to integrate with that in some way That's much better. It took me a few iterations to get down to an appropriate number of threads where it wouldn't fail, but being able to do it

jenkins deploy snapshots

Can we configure jenkins to deploy snapshots on every build? I assume there is an apache snapshots repository that we can use for this purpose. Maybe this was previously turned off for some reason that I am not aware of. It's somewhat annoying to build the whole project when i am making changes in

Re: [VOTE] Accumulo Instamo Archetype 1.4.4-RC3

The released tarball has a DEPENDENCIES file that is not in source control. I think we talked about this last time, but wanted to confirm that you were aware of this. All other files match. I don't think this is an issue, since iirc that file is auto-gen'd. Signature on the source release matches

Re: [VOTE] Accumulo Instamo Archetype 1.4.4-RC3

Thanks for the information either way, Mike. > > > On 10/13/2013 12:29 AM, Mike Drob wrote: >> >> There is no archetype-catalog.xml file deployed, so I have to mvn install >> before I am able to generate projects using the archetype. After I install >> however, everything works fine. > >

Re: Hadoop 2.0 Support for Accumulo 1.4 Branch

Responses Inline. - Mike On Mon, Oct 14, 2013 at 12:55 PM, Sean Busbey wrote: > Hey All, > > I'd like to restart the conversation from end July / start August about > Hadoop 2 support on the 1.4 branch. > > Specifically, I'd like to get some requirements ironed out so I can file > one or more j

Re: Contributions from github pull requests and license grant

This discussion [1] from legal-discuss seems to be the most recent word regarding github pull requests, and looks like they are fair game. [1]: http://markmail.org/thread/3gvsg4qgc2khve27 On Tue, Oct 15, 2013 at 6:34 PM, Billie Rinaldi wrote: > On Tue, Oct 15, 2013 at 1:45 PM, Christopher wrot

Re: linking to the repos for Accumulo contrib projects

New page and also linked from the source page makes the most sense to me. On Oct 15, 2013 1:20 AM, "Sean Busbey" wrote: > Hiya! > > I was just looking for the current wikisearch example and had some > difficulty. I ended up tracking through the INFRA ticket to switch to > git[1] followed by looki

Re: Review Request 14844: ACCUMULO-1789 Add timeout scaling factor to tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14844/#review27312 --- Ship it! Ship It! - Mike Drob On Oct. 22, 2013, 5:26 p.m., Bill

Re: Review Request 15002: ACCUMULO-1307 - adds flag to prompt fate threads to end

ver/src/main/java/org/apache/accumulo/server/master/Master.java <https://reviews.apache.org/r/15002/#comment53775> If you're addressing this TODO, can you remove it from the code? - Mike Drob On Oct. 29, 2013, 3:52

Re: Review Request 15002: ACCUMULO-1307 - adds flag to prompt fate threads to end

Did update #3 somehow eat update #2? It looks like the AtomicBoolean code got removed. On Tue, Oct 29, 2013 at 12:28 PM, John Vines wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r

Re: git commit: ACCUMULO-1507 - adding a (nonfunctional/commented out) test for the monitor issue

Does this test compile? If so, I'd like to see it uncommented and annotated with @Ignore. If not, I'd like to see a more surgical comment block, instead of taking a hammer to the whole method. Code like this is very easy to forget about down the read, whereas an ignored test shows up in reports and

Re: Review Request 15166: ACCUMULO-802 Tablespaces

core/src/main/java/org/apache/accumulo/core/client/impl/Tables.java <https://reviews.apache.org/r/15166/#comment55389> No. - Mike Drob On Nov. 1, 2013, 2:01 a.m., Christopher Tubbs wrote: > > --- > This is

Re: Review Request 15166: ACCUMULO-802 Tablespaces

I'm having trouble with Review Board and cannot load pages 2-5 of the diff. Is anybody else seeing experiencing this? On Fri, Nov 8, 2013 at 2:08 PM, Mike Drob wrote: > > --- > This is an automatically generated e-mail. T

Re: mvn dependency:analyze

+1 Nobody looks at dependency resolution until it bites you in the rear. We could conceivably add this to the pom to automate the running, but would still need to make sure that somebody reads the output. On Fri, Nov 8, 2013 at 10:37 AM, Billie Rinaldi wrote: > I would like to add running mvn

Re: mvn dependency:analyze

I feel like that would be very flaky. Reflection can cause some weird dependency graphs that maven is not able to follow. On Fri, Nov 8, 2013 at 2:32 PM, Sean Busbey wrote: > On Fri, Nov 8, 2013 at 1:29 PM, Mike Drob wrote: > > > +1 > > > > Nobody looks at dependency r

Re: [VOTE] add mvn dependency:analyze to release process

+1 On Nov 8, 2013 3:12 PM, "Christopher" wrote: > +1 for an explicit check to be added to the release process. > (however, these dependency issues should really be regularly checked > and considered throughout development whenever dependencies are > added/removed). > > -- > Christopher L Tubbs II

Re: Review Request 15166: ACCUMULO-802 Tablespaces

> On Nov. 8, 2013, 7:08 p.m., Mike Drob wrote: > > core/src/main/java/org/apache/accumulo/core/client/TableNamespaceExistsException.java, > > line 24 > > <https://reviews.apache.org/r/15166/diff/1/?file=376077#file376077line24> > > > > I would like to

Re: Review Request 15166: ACCUMULO-802 Tablespaces

- Mike --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15166/#review28554 --- On Nov. 1, 2013, 2:01 a.m., Christopher Tubbs wrote: > >

Re: Review Request 15002: ACCUMULO-1307 - adds flag to prompt fate threads to end

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15002/#review28666 --- Ship it! Ship It! - Mike Drob On Oct. 29, 2013, 5:02 p.m., John

Re: integration tests

Do you get the same failures without the fork count option? On Nov 12, 2013 8:30 PM, "David Medinets" wrote: > With Accumulo v1.5 I ran "mvn clean verify -Daccumulo.it.forkCount=12". The > result was 30 failures in the Testing module. Is this OBE or significant? > > Failed tests: interpreter(or

Re: FooConstraint.jar Source

Does not having the source run us afoul of ASF policy? On Wed, Nov 20, 2013 at 12:25 PM, Josh Elser wrote: > Yup to both. We definitely need to address this before we make the next > release of Accumulo. > > > On 11/20/13, 11:44 AM, Keith Turner wrote: > >> On Wed, Nov 20, 2013 at 11:33 AM, Sea

Re: Discussion on Namespace permissions

It is absolutely useful to grant users full access within a namespace and no access outside of it. If I'm running some experiments then let me (and possibly the rest of my team) create and drop to heart's content in our own personal area and don't worry about the rest of the system. Having a permi

Re: Review Request 16187: ACCUMULO-1984 - Clean up Instance closing

/ZooReader.java <https://reviews.apache.org/r/16187/#comment57826> Add comments to caution users about the possibility of interruption. - Mike Drob On Dec. 11, 2013, 6:52 p.m., Bill Havanki wrote: > > --- > This is an automatica

Review Request 16196: ACCUMULO-1896

are correct. Thanks, Mike Drob

Re: Review Request 16196: ACCUMULO-1896

utomatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16196/#review30239 --- On Dec. 11, 2013, 11:18 p.m., Mike Drob wrote: > > --- > This is an autom

Re: Review Request 16196: ACCUMULO-1896

> On Dec. 12, 2013, 2:11 a.m., Sean Busbey wrote: > > pom.xml, lines 315-442 > > <https://reviews.apache.org/r/16196/diff/1/?file=396796#file396796line315> > > > > Please leave out the move of unrelated pom sections. > > Mike Drob wrote: > Ugh

Review Request 16217: ACCUMULO-1868 add missing directories to assembly tarball

logs|lib\/ext" Thanks, Mike Drob

Re: Review Request 16224: ACCUMULO-1986 - data integrity checks for Key and Mutation Thrift constructors

qualify references to if as this.assertEquals(...). Not sure which one is preferable. I don't like the name "checkCU" but can't offer any alternatives, so maybe it's fine. - Mike Drob On Dec. 12, 2013, 9:32 p.m., Bill Havanki wrote: > > ---

Re: Resource leak warnings

Comments Inline. On Fri, Dec 13, 2013 at 12:44 PM, Christopher wrote: > What should we do about all these additional resource leak warnings > added as a result of ACCUMULO-1984? (ACCUMULO-2010) > > As I see it, there's a few options: > > 0. Revert the previous patch for ACCUMULO-1984 > 1. Ignor

Re: Review Request 16087: ACCUMULO-1796 - document Hadoop versions supported by 1.4.x binary

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16087/#review30376 --- Ship it! Ship It! - Mike Drob On Dec. 9, 2013, 5:35 p.m., Bill

Re: Review Request 16388: ACCUMULO-2042 - fix scalability test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16388/#review30717 --- Ship it! Ship It! - Mike Drob On Dec. 19, 2013, 5:31 p.m., Bill

Re: Resource leak warnings

This conversation is way past dead at this point, but I wanted to respond now that things have had a time to cool off. I didn't really pursue a response last week because I felt like you [Christopher] had unilaterally imposed your will on the rest of us and were unwilling to discuss things further.

Review Request 16427: ACCUMULO-2076 add Instance builder based on classpath resources

/accumulo/core/client/ZooKeeperInstance.java fcadd384cc150b8422c3f5c33970c26efe142132 Diff: https://reviews.apache.org/r/16427/diff/ Testing --- None, yet. :) Thanks, Mike Drob

Re: Resource leak warnings

Keith, I'm not sure I understand the alternative solution that you and Christopher discussed. Can you explain it in a bit more detail, please? I have my own interpretation of what I _think_ you were proposing, but I'd rather not risk putting words in your mouth. Specifically, I'm interested in the

Re: Resource leak warnings

oses all zookeepers, and does not allow any more to be created > ZooSession.shutdownNow(); > } > } > > > On Mon, Dec 23, 2013 at 4:36 PM, Mike Drob wrote: > > > Keith, > > > > I'm not sure I understand the alternative solution that you and > Christo

Re: Review Request 16427: ACCUMULO-2076 add Instance builder based on classpath resources

rated e-mail. To reply, visit: https://reviews.apache.org/r/16427/#review30800 --- On Dec. 20, 2013, 9:52 p.m., Mike Drob wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.or

Re: Review Request 16427: ACCUMULO-2076 add Instance builder based on classpath resources

/main/java/org/apache/accumulo/core/client/ZooKeeperInstance.java fcadd384cc150b8422c3f5c33970c26efe142132 Diff: https://reviews.apache.org/r/16427/diff/ Testing --- None, yet. :) Thanks, Mike Drob

Re: Review Request 16427: ACCUMULO-2076 add Instance builder based on classpath resources

------ On Dec. 20, 2013, 9:52 p.m., Mike Drob wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16427/ >

Re: Review Request 16427: ACCUMULO-2076 add Instance builder based on classpath resources

/ Testing --- None, yet. :) Thanks, Mike Drob

Formatter - White Space Issue?

Accumulators, I'm using the provided formatter from our site[1], but it looks like it leaves trailing white space in java doc comments[2]. Has anybody else seen this, or does anybody have a fix? Mike [1]: https://git-wip-us.apache.org/repos/asf?p=accumulo.git;a=tree;f=contrib;hb=HEAD [2]: https:

Re: Review Request 16485: ACCUMULO-2103 - Better ZooSession management

ookeeper/ZooSession.java <https://reviews.apache.org/r/16485/#comment59158> Can this leak unclosed sessions? src/core/src/test/java/org/apache/accumulo/core/zookeeper/ZooSessionTest.java <https://reviews.apache.org/r/16485/#comment59159> Should verify that it didn't clo

Re: Let's have a round of maintenance releases

+1 I'm willing to take point on the 1.4.5 RCs. On Fri, Jan 3, 2014 at 8:50 AM, Josh Elser wrote: > On 1/3/14, 11:34 AM, Sean Busbey wrote: > >> We're past the end of 2013 now and while there's been good progress on >> polishing up 1.6.0, it looks like there's still a good deal of work before >

  1   2   3   4   5   6   >