RE: [DISCUSS] Enable Github wiki in asf.yaml?

2023-02-28 Thread dlmarion
ki. If we need to store documents in a version controlled > way, we can store them on the website, or in our project's SVN dev > space. The wiki is just another place people would have to follow if > they want to participate, and I don't think that serves us. Therefore, > I think we shouldn

[DISCUSS] Enable Github wiki in asf.yaml?

2023-02-25 Thread dlmarion
space. The wiki is just another place people would have to follow if they want to participate, and I don't think that serves us. Therefore, I think we shouldn't use it. On Fri, Feb 24, 2023, 15:59 wrote: > This is an automated email from the ASF dual-hosted git repository. > > dlmari

RE: Slack call notes

2020-03-25 Thread dlmarion
I couldn't make the call today, but am curious if anyone has previously brought up creating a FileSystem API for Accumulo so that we could use implementations other than Hadoop. I realize that Hadoop provides implementations for things other than HDFS but that doesn't necessarily mean that

RE: accumulo.metadata table online but scans hang

2017-08-30 Thread dlmarion
Re #2: Does your Accumulo processes run as the hdfs user on the O/S, or as the accumulo user? Make sure you are checking the correct users trash folder. Also, check the Accumulo garbage collector log to see if the GC process deleted the WAL files. Take a look at [1] to see if you are hitting

RE: [DISCUSS] Periodic table exports

2017-07-14 Thread dlmarion
portion of it worked to some degree. I have not touched it in two years, but it could be a starting point or someone could use the ideas behind it to do something on their own. https://github.com/dlmarion/raccovery -Original Message- From: Christopher [mailto:ctubb...@apache.org] Sent

RE: New committer/PMC: Ivan Bella

2017-07-13 Thread dlmarion
Congrats Ivan! -Original Message- From: Christopher [mailto:ctubb...@apache.org] Sent: Thursday, July 13, 2017 3:02 PM To: Accumulo Dev List Subject: New committer/PMC: Ivan Bella The Project Management Committee (PMC) for Apache Accumulo has invited Ivan

[GitHub] accumulo issue #272: ACCUMULO-4654 Added the ability to continue balancing e...

2017-06-26 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/272 Latest commit causing build to fail with test failure at: HostRegexTableLoadBalancerReconfigurationTest.testConfigurationChanges:83 expected:<5> but was:<0> --- If your pr

[GitHub] accumulo pull request #272: ACCUMULO-4654 Added the ability to continue bala...

2017-06-26 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/272#discussion_r124015943 --- Diff: server/base/src/main/java/org/apache/accumulo/server/master/balancer/HostRegexTableLoadBalancer.java --- @@ -66,6 +71,9 @@ * It's

[GitHub] accumulo issue #272: ACCUMULO-4654 Added the ability to continue balancing e...

2017-06-22 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/272 Agreed that the default should be the old behavior in the 1.x releases. We can change the behavior in the 2.x release line and it should be noted in the 2.0 release notes. --- If your project

[GitHub] accumulo pull request #272: ACCUMULO-4654 Added the ability to continue bala...

2017-06-20 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/272#discussion_r123022077 --- Diff: server/base/src/main/java/org/apache/accumulo/server/master/balancer/HostRegexTableLoadBalancer.java --- @@ -81,15 +86,24

[GitHub] accumulo pull request #272: ACCUMULO-4654 Added the ability to continue bala...

2017-06-20 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/272#discussion_r123021060 --- Diff: server/base/src/main/java/org/apache/accumulo/server/master/balancer/HostRegexTableLoadBalancer.java --- @@ -81,15 +86,24

[GitHub] accumulo pull request #270: ACCUMULO-4657 Reduce excessive logging / logging...

2017-06-20 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/270#discussion_r123020151 --- Diff: server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java --- @@ -894,12 +895,19 @@ public static void

[GitHub] accumulo pull request #270: ACCUMULO-4657 Reduce excessive logging / logging...

2017-06-20 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/270#discussion_r122974876 --- Diff: server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java --- @@ -894,12 +895,19 @@ public static void

[GitHub] accumulo pull request #270: ACCUMULO-4657 Reduce excessive logging / logging...

2017-06-19 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/270#discussion_r122793218 --- Diff: server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java --- @@ -894,12 +894,19 @@ public static void

[GitHub] accumulo issue #269: ACCUMULO-4656

2017-06-19 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/269 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] accumulo issue #265: ACCUMULO-4647 - start of contributor guide

2017-06-06 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/265 Merged manually --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] accumulo pull request #265: ACCUMULO-4647 - start of contributor guide

2017-06-06 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/265#discussion_r120398028 --- Diff: CONTRIBUTING.md --- @@ -0,0 +1,43 @@ + + +# Contributors Guide + + If you believe that you have found a bug, please search

[GitHub] accumulo pull request #265: ACCUMULO-4647 - start of contributor guide

2017-06-06 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/265#discussion_r120391771 --- Diff: CONTRIBUTING.md --- @@ -0,0 +1,43 @@ + + +# Contributors Guide + + If you believe that you have found a bug, please search

[GitHub] accumulo issue #265: ACCUMULO-4647 - start of contributor guide

2017-06-06 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/265 Made modifications based on comments from @mikewalch and @joshelser. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] accumulo pull request #265: ACCUMULO-4647 - start of contributor guide

2017-06-06 Thread dlmarion
GitHub user dlmarion opened a pull request: https://github.com/apache/accumulo/pull/265 ACCUMULO-4647 - start of contributor guide You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/accumulo ACCUMULO-4647 Alternatively you

[GitHub] accumulo pull request #260: ACCUMULO-4643 initial implementation

2017-06-04 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/260#discussion_r120024937 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java --- @@ -746,16 +774,29 @@ Batch nextBatch(SortedKeyValueIterator

[GitHub] accumulo issue #260: ACCUMULO-4643 initial implementation

2017-06-01 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/260 @joshelser Is there an example of creating a new metric, or is it still all JMX based? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] accumulo pull request #260: ACCUMULO-4643 initial implementation

2017-06-01 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/260#discussion_r119692389 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java --- @@ -586,6 +591,11 @@ private LookupResult lookup

[GitHub] accumulo issue #260: ACCUMULO-4643 initial implementation

2017-06-01 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/260 LGTM, might need more tests for the edge cases. May also want to look at the iterator test harness to see if it needs to be modified. --- If your project is set up for it, you can reply

[GitHub] accumulo pull request #260: ACCUMULO-4643 initial implementation

2017-06-01 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/260#discussion_r119597285 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java --- @@ -746,16 +764,22 @@ Batch nextBatch(SortedKeyValueIterator

[GitHub] accumulo pull request #260: ACCUMULO-4643 initial implementation

2017-06-01 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/260#discussion_r119597061 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java --- @@ -586,6 +591,11 @@ private LookupResult lookup

[GitHub] accumulo pull request #260: ACCUMULO-4643 initial implementation

2017-05-31 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/260#discussion_r119457446 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/SortedKeyValueIterator.java --- @@ -147,4 +147,35

[GitHub] accumulo pull request #260: ACCUMULO-4643 initial implementation

2017-05-31 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/260#discussion_r119457222 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/SortedKeyValueIterator.java --- @@ -147,4 +147,35

[GitHub] accumulo issue #261: ACCUMULO-4644 Removed AccumuloConfiguration from BlockC...

2017-05-30 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/261 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] accumulo issue #256: ACCUMULO-4463: Make block cache implementation configur...

2017-05-22 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/256 Merged manually --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-22 Thread dlmarion
Github user dlmarion closed the pull request at: https://github.com/apache/accumulo/pull/256 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] accumulo issue #256: ACCUMULO-4463: Make block cache implementation configur...

2017-05-19 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/256 yes please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] accumulo issue #256: ACCUMULO-4463: Make block cache implementation configur...

2017-05-18 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/256 @keith-turner looks good. Is the Builder just for tests? I'm assuming you don't need it in TabletServerResourceManager, RFileScanner, etc because you already have the configuration. Feel free

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-12 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r116310180 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServerResourceManager.java --- @@ -169,25 +170,23 @@ public

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-12 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r116305691 --- Diff: core/src/main/java/org/apache/accumulo/core/file/blockfile/cache/BlockCacheManager.java --- @@ -0,0 +1,121 @@ +/* + * Licensed

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-12 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r116305578 --- Diff: core/src/main/java/org/apache/accumulo/core/file/blockfile/cache/BlockCacheManager.java --- @@ -0,0 +1,121 @@ +/* + * Licensed

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-12 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r116305410 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFileScanner.java --- @@ -135,15 +139,34 @@ public long requestCount

[GitHub] accumulo issue #256: ACCUMULO-4463: Make block cache implementation configur...

2017-05-12 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/256 @keith-turner @mjwall - I added a new commit that takes into account your comments and changes --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-12 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r116251656 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFileScanner.java --- @@ -135,16 +142,41 @@ public long requestCount

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-12 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r116232168 --- Diff: core/src/main/java/org/apache/accumulo/core/file/blockfile/cache/BlockCacheManager.java --- @@ -0,0 +1,101 @@ +/* + * Licensed

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-12 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r116231739 --- Diff: core/src/main/java/org/apache/accumulo/core/file/blockfile/cache/BlockCacheManager.java --- @@ -0,0 +1,101 @@ +/* + * Licensed

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implementation c...

2017-05-09 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r115597102 --- Diff: core/src/main/java/org/apache/accumulo/core/file/blockfile/cache/BlockCacheConfiguration.java --- @@ -0,0 +1,92 @@ +/* + * Licensed

[GitHub] accumulo issue #256: ACCUMULO-4463: Make block cache implementation configur...

2017-05-09 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/256 I made some significant changes in this latest commit. @joshelser , @keith-turner - I believe I addressed most, if not all, of your previous comments. --- If your project is set up for it, you

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implentation con...

2017-05-08 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r115355361 --- Diff: core/src/main/java/org/apache/accumulo/core/file/blockfile/cache/BlockCacheFactory.java --- @@ -0,0 +1,33 @@ +/* + * Licensed

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implentation con...

2017-05-08 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r115331908 --- Diff: core/src/main/java/org/apache/accumulo/core/conf/Property.java --- @@ -245,7 +245,8 @@ TSERV_PREFIX("tserver.", null, PropertyT

[GitHub] accumulo issue #256: ACCUMULO-4463: Make block cache implentation configurab...

2017-05-08 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/256 Ok, let me see if I can rework this given the comments up to this point. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implentation con...

2017-05-08 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r115288967 --- Diff: core/src/main/java/org/apache/accumulo/core/file/blockfile/cache/BlockCache.java --- @@ -17,10 +17,30 @@ */ package

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implentation con...

2017-05-08 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/256#discussion_r115287907 --- Diff: core/src/main/java/org/apache/accumulo/core/conf/Property.java --- @@ -245,7 +245,8 @@ TSERV_PREFIX("tserver.", null, PropertyT

[GitHub] accumulo pull request #256: ACCUMULO-4463: Make block cache implentation con...

2017-05-08 Thread dlmarion
GitHub user dlmarion opened a pull request: https://github.com/apache/accumulo/pull/256 ACCUMULO-4463: Make block cache implentation configurable You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/accumulo ACCUMULO-4463

[GitHub] accumulo issue #199: DO NOT MERGE: PROOF OF CONCEPT

2017-01-09 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/199 @phrocker - Just thinking about multi-tenancy and competition for resources, allowing an administrator to define upper limits, and making sure that user operations don't trump an administrator

[GitHub] accumulo issue #199: DO NOT MERGE: PROOF OF CONCEPT

2017-01-09 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/199 I see in Property.java the definition of queues / pools. Are the high water marks for the requested resources set there also? I could not tell from the PR, but I think an admin should be able

[GitHub] accumulo issue #193: [ACCUMULO-4535] Fix NPE in HostRegexTableLoadBalancer

2017-01-04 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/193 Thanks @joshelser . I have not had time yet to do it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] accumulo issue #193: [ACCUMULO-4535] Fix NPE in HostRegexTableLoadBalancer

2017-01-03 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/193 I was hoping that someone else would look at this and +1 it before merging it. What's your timeline on needing this done? --- If your project is set up for it, you can reply to this email

[GitHub] accumulo issue #193: [ACCUMULO-4535] Fix NPE in HostRegexTableLoadBalancer

2016-12-15 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/193 +1 - that was easy --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] accumulo issue #176: Wrote blog post about durability and performance

2016-10-31 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/176 ACCUMULO-4000 has some information too. It's not always seen during decommissioning though. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] accumulo issue #176: Wrote blog post about durability and performance

2016-10-31 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/176 > I'm not sure I understand the reasoning behind that logic (admittedly, it's "early"). Specifically, how would a failure of the local node be any different than a failure of a

[GitHub] accumulo issue #176: Wrote blog post about durability and performance

2016-10-31 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/176 May want to point out https://issues.apache.org/jira/browse/HDFS-3702 as an option starting with Hadoop 2.8, so that if the node goes down and can't be brought back up, you can still recover

[GitHub] accumulo issue #173: Updated projects page

2016-10-27 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/173 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[ANNOUNCE] Timely 0.0.3 Released

2016-10-27 Thread dlmarion
For those interested, we released the third version of Timely today which includes several feature changes and performance improvements. Release notes are at: https://nationalsecurityagency.github.io/timely/docs/#release-notes

RE: [DISCUSS] Would a visibility histogram on a table be harmful?

2016-10-11 Thread dlmarion
So, to get the set of visibilities used in a table, we would have to open all of the rfiles? > -Original Message- > From: Dylan Hutchison [mailto:dhutc...@cs.washington.edu] > Sent: Tuesday, October 11, 2016 3:43 PM > To: Accumulo Dev List > Subject: Re: [DISCUSS] Would a visibility

Re: 1.8.0 RC3

2016-08-26 Thread dlmarion
Christopher re-opened 3948 and said he was testing a patch. - Original Message - From: "Michael Wall" To: dev@accumulo.apache.org Sent: Friday, August 26, 2016 9:05:34 AM Subject: 1.8.0 RC3 I was going to make another RC for 1.8.0, but we have open 4 tickets.

Re: [VOTE] Plan for next release

2016-08-23 Thread dlmarion
Agreed - this thread should be left to votes only. Let it pass or fail. - Original Message - From: "Josh Elser" To: dev@accumulo.apache.org Sent: Tuesday, August 23, 2016 8:15:01 AM Subject: Re: [VOTE] Plan for next release If we really need to keep arguing

RE: [VOTE] Plan for next release

2016-08-22 Thread dlmarion
I share your concerns and have proposed releasing a 1.8.0 as-is, followed by a 2.0 with much the same artifacts plus Java 8 source. In talking with Christopher about this though, that means that the community would be supporting 1.6 (until 1.6.6 is released), 1.7.x, 1.8.x, and 2.0.x. Being on

RE: [VOTE] Plan for next release

2016-08-22 Thread dlmarion
+1 Additionally, I'm also +1 for removing deprecated items. Leaving them in will cause more code to be modified as bugs are fixed and features added. Let's rip the band-aid off and move forward. > -Original Message- > From: Christopher [mailto:ctubb...@apache.org] > Sent: Monday,

Re: [VOTE] Accumulo 1.8.0-rc2

2016-08-16 Thread dlmarion
+0 (because I had issues running mvn verify. I kept running out of memory at the tests module even with Xmx of 2560M) I did: verify the sha1 and md5 values verified semver compliance with Josh using the semver compliance plugin (ACCUMULO-4307) - Original Message - From: "Michael

[GitHub] accumulo issue #139: Log naming

2016-08-15 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/139 Yeah, no worries, go for it. Related, are you finding any issues with running multiple tservers? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] accumulo issue #139: Log naming

2016-08-15 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/139 Changes LGTM. I should be able to check them out more thoroughly tomorrow. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] accumulo issue #138: ACCUMULO-4406: Add instance number to .out and .err fil...

2016-08-15 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/138 No, not what I expected. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] accumulo issue #138: ACCUMULO-4406: Add instance number to .out and .err fil...

2016-08-15 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/138 Please do. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] accumulo pull request #138: ACCUMULO-4406: Add instance number to .out and ....

2016-08-15 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/138#discussion_r74800933 --- Diff: assemble/bin/start-daemon.sh --- @@ -103,6 +103,9 @@ if [[ "$SERVICE" != "tserver" || $NUM_TSERVERS -eq 1 ]]; then

[GitHub] accumulo pull request #138: ACCUMULO-4406: Add instance number to .out and ....

2016-08-15 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/138#discussion_r74798567 --- Diff: assemble/bin/start-daemon.sh --- @@ -103,6 +103,9 @@ if [[ "$SERVICE" != "tserver" || $NUM_TSERVERS -eq 1 ]]; then

[GitHub] accumulo pull request #138: ACCUMULO-4406: Add instance number to .out and ....

2016-08-15 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/138#discussion_r74797105 --- Diff: assemble/bin/start-daemon.sh --- @@ -103,6 +103,9 @@ if [[ "$SERVICE" != "tserver" || $NUM_TSERVERS -eq 1 ]]; then

[GitHub] accumulo pull request #138: ACCUMULO-4406: Add instance number to .out and ....

2016-08-15 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/138#discussion_r74796945 --- Diff: assemble/bin/start-daemon.sh --- @@ -103,6 +103,9 @@ if [[ "$SERVICE" != "tserver" || $NUM_TSERVERS -eq 1 ]]; then

[GitHub] accumulo issue #138: ACCUMULO-4406: Add instance number to .out and .err fil...

2016-08-15 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/138 Cool, thanks @joshelser --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] accumulo issue #137: ACCUMULO-4406: Add instance number to .out and .err fil...

2016-08-15 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/137 Closed as target branch was incorrect. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] accumulo pull request #138: ACCUMULO-4406: Add instance number to .out and ....

2016-08-15 Thread dlmarion
GitHub user dlmarion opened a pull request: https://github.com/apache/accumulo/pull/138 ACCUMULO-4406: Add instance number to .out and .err files @mjwall @joshelser You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/accumulo

[GitHub] accumulo pull request #137: ACCUMULO-4406: Add instance number to .out and ....

2016-08-15 Thread dlmarion
Github user dlmarion closed the pull request at: https://github.com/apache/accumulo/pull/137 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] accumulo pull request #137: ACCUMULO-4406: Add instance number to .out and ....

2016-08-15 Thread dlmarion
GitHub user dlmarion opened a pull request: https://github.com/apache/accumulo/pull/137 ACCUMULO-4406: Add instance number to .out and .err files @mjwall @joshelser You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/accumulo

Re: Change in out-of-the-box log file naming after ACCUMULO-4328

2016-08-15 Thread dlmarion
The change to add the instance number to the .out and .err files should be pretty simple. I'll take a look at it. Regarding the change if there is only one tserver, I'm not sure how you would specify the conditional in the logging configuration file. - Original Message - From: "Josh

[GitHub] accumulo issue #134: ACCUMULO-4391 Added synchronization around the availabl...

2016-07-28 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/134 +1 LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

Re: [OT] Can no longer obtain JDK6/JDK7 tarballs

2016-07-20 Thread dlmarion
Can't you use JDK8 and keep the source and target at 1.7? - Original Message - From: "Christopher" To: "Accumulo Dev List" Sent: Tuesday, July 19, 2016 6:22:58 PM Subject: [OT] Can no longer obtain JDK6/JDK7 tarballs I know we've

[GitHub] accumulo issue #125: ACCUMULO-4375 Add missing byte[]-based Key constructors

2016-07-16 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/125 LGTM. @mjwall - can we get this into the 1.8 release? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] accumulo issue #124: ACCUMULO-4375 Add missing byte[]-based Key constructors

2016-07-16 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/124 You might want to rebase this onto the 1.8 branch. If we have consensus, then we might be able to pull this into the 1.8 release. --- If your project is set up for it, you can reply

Re: Board report draft July 2016

2016-07-12 Thread dlmarion
+1 LGTM. - Original Message - From: "Billie Rinaldi" To: "Accumulo Dev List" Sent: Tuesday, July 12, 2016 12:48:51 PM Subject: Board report draft July 2016 Hello Apache Accumulo community, The Apache Accumulo PMC has decided to start

Re: Publicity/blog on Timely

2016-06-22 Thread dlmarion
Never heard of Warp 10. Do you have a link? - Original Message - From: "Mathias Herberts" To: dev@accumulo.apache.org Sent: Wednesday, June 22, 2016 4:23:36 PM Subject: Re: Publicity/blog on Timely Have you considered starting with Warp 10 instead of

Re: [ANNOUNCE] Timely - Secure Time Series Database

2016-06-22 Thread dlmarion
In the Grafana U/I you can specify a set of tags for the metric query. We take the first tag and seek to it, then take the remaining tags in the query and filter on them. Specifying the tags in the metric query in order of more specific to less specific will give better performance. I thought

Re: Publicity/blog on Timely

2016-06-22 Thread dlmarion
We have discussed giving a presentation at the Accumulo Summit, nothing beyond that at the moment though. - Original Message - From: "Josh Elser" To: dev@accumulo.apache.org Sent: Wednesday, June 22, 2016 3:28:48 PM Subject: Publicity/blog on Timely My

[ANNOUNCE] Timely - Secure Time Series Database

2016-06-22 Thread dlmarion
Timely is a time series database application that provides secure access to time series data. It is designed to be used with Apache Accumulo for persistence and Grafana for visualization. Timely is located at https://github.com/NationalSecurityAgency/timely .

[GitHub] accumulo pull request #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/111#discussion_r67013512 --- Diff: core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java --- @@ -262,4 +267,46 @@ public boolean apply(final String input

[GitHub] accumulo pull request #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/111#discussion_r67011320 --- Diff: core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java --- @@ -262,4 +267,46 @@ public boolean apply(final String input

[GitHub] accumulo pull request #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/111#discussion_r67008218 --- Diff: core/src/test/java/org/apache/accumulo/core/conf/AccumuloConfigurationTest.java --- @@ -72,4 +72,79 @@ public void testGetPropertyByString

[GitHub] accumulo pull request #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/111#discussion_r67004086 --- Diff: core/src/test/java/org/apache/accumulo/core/conf/AccumuloConfigurationTest.java --- @@ -72,4 +72,79 @@ public void testGetPropertyByString

[GitHub] accumulo pull request #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/111#discussion_r67003317 --- Diff: server/base/src/main/java/org/apache/accumulo/server/util/Admin.java --- @@ -374,15 +374,17 @@ private static void stopTabletServer(final

[GitHub] accumulo pull request #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion commented on a diff in the pull request: https://github.com/apache/accumulo/pull/111#discussion_r67002968 --- Diff: core/src/test/java/org/apache/accumulo/core/conf/AccumuloConfigurationTest.java --- @@ -72,4 +72,79 @@ public void testGetPropertyByString

[GitHub] accumulo issue #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/111 Will apply manually. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] accumulo pull request #111: ACCUMULO-4331: first draft

2016-06-14 Thread dlmarion
Github user dlmarion closed the pull request at: https://github.com/apache/accumulo/pull/111 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] accumulo pull request #113: ACCUMULO-4341: Preload classes

2016-06-14 Thread dlmarion
Github user dlmarion closed the pull request at: https://github.com/apache/accumulo/pull/113 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] accumulo issue #113: ACCUMULO-4341: Preload classes

2016-06-14 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/113 Will apply this manually. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] accumulo pull request #113: ACCUMULO-4341: Preload classes

2016-06-13 Thread dlmarion
GitHub user dlmarion opened a pull request: https://github.com/apache/accumulo/pull/113 ACCUMULO-4341: Preload classes You can merge this pull request into a Git repository by running: $ git pull https://github.com/dlmarion/accumulo ACCUMULO-4341 Alternatively you can review

[GitHub] accumulo issue #111: ACCUMULO-4331: first draft

2016-06-13 Thread dlmarion
Github user dlmarion commented on the issue: https://github.com/apache/accumulo/pull/111 FWIW, with this latest change I was able to start Accumulo to include two tablet servers with setting all of the *.port.client properties to the range of 11000-11006. --- If your project is set

  1   2   3   >