Re: [GitHub] [accumulo-access] keith-turner opened a new pull request, #2: adds asf yaml file

2023-09-07 Thread Keith Turner
I overlooked an option for setting the email when creating the new git repository. Hopefully this PR will fix it. On Thu, Sep 7, 2023 at 4:32 PM keith-turner (via GitHub) wrote: > > keith-turner opened a new pull request, #2: > URL: https://github.com/apache/accumulo-access/pull/2 > >(no co

[GitHub] [accumulo-access] keith-turner opened a new pull request, #2: adds asf yaml file

2023-09-07 Thread via GitHub
keith-turner opened a new pull request, #2: URL: https://github.com/apache/accumulo-access/pull/2 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscrib

[GitHub] [accumulo-access] keith-turner opened a new pull request, #1: improves unit test coverage

2023-09-07 Thread via GitHub
keith-turner opened a new pull request, #1: URL: https://github.com/apache/accumulo-access/pull/1 Did the following for this commit. * Looked at Accumulo existing test and brough test cases over * Looked at the specification and thought of new cases to cover * Looked at the re

Re: New Accumulo git repository

2023-09-07 Thread Keith Turner
Created https://github.com/apache/accumulo-access with a subset of the changes from 3715. For the rest of the changes in 3715 I am contemplating creating a branch in accumulo called accumulo-access where accumulo is modified to use a snapshot version of this library. This will allow collaboration

Re: New Accumulo git repository

2023-09-07 Thread Keith Turner
On Thu, Sep 7, 2023 at 8:53 AM Marc P. wrote: > Out of curiosity, what is the benefit of having a new repo over a release > target of accumulo-visibility ? > The main benefit of a new repo is that it allows it to release independently of other parts of Accumulo with its own version number. In #

Re: New Accumulo git repository

2023-09-07 Thread Christopher
I'm not 100% certain about the name, but a new repo is a really low risk thing. I say go for it. On Thu, Sep 7, 2023, 08:54 Marc P. wrote: > Out of curiosity, what is the benefit of having a new repo over a release > target of accumulo-visibility ? > > I've used VisibilityEvaluator on non Accumu

Re: New Accumulo git repository

2023-09-07 Thread Marc P.
Out of curiosity, what is the benefit of having a new repo over a release target of accumulo-visibility ? I've used VisibilityEvaluator on non Accumulo projects, so I can see some perceived benefit, but in those projects I wanted to support a visibility expression or ColumnVisibility "as Accumulo

Re: New Accumulo git repository

2023-09-07 Thread Dave Marion
I don't think the risk of not creating a release from the new repo exists. I believe we have to use non-snapshot dependencies when creating an Accumulo release. Right? I have no issue with creating the repo now and no issue with the name, since we can rename it. On Wed, Sep 6, 2023 at 3:18 PM Keit