Skip to content

Fixing error with mistaking volume for directory.#2046

Merged
ctubbsii merged 3 commits intoapache:mainfrom
jjmar11:issue_2002
May 12, 2021
Merged

Fixing error with mistaking volume for directory.#2046
ctubbsii merged 3 commits intoapache:mainfrom
jjmar11:issue_2002

Conversation

@jjmar11
Copy link
Contributor

@jjmar11 jjmar11 commented Apr 29, 2021

Can volume just have "tables" as one of its base elements?

So for example: ""file:///some/path/to/accumulo/tables/tables" in which the last "tables" is the actual directory.

@Manno15
Copy link
Contributor

Manno15 commented Apr 29, 2021

It appears the build has failed due to improper formatting If you run something like: mvn clean package -DskipTests locally, it should apply our formatter. Then you can commit and push those changes to run the QA build.

@ctubbsii ctubbsii linked an issue May 1, 2021 that may be closed by this pull request
@ctubbsii
Copy link
Member

Thanks for the PR @jjmar11 . Apologies for the delay in reviewing this. If you wish to be added as a contributor to https://accumulo.apache.org/people/ , please open a pull request to add yourself at https://github.com/apache/accumulo-website/edit/main/pages/people.md and leave a reference to apache/accumulo#2046 in your commit log.

If you intend to be a regular contributor to Accumulo projects, please consider subscribing to our developer mailing list (https://accumulo.apache.org/contact-us/) and introducing yourself. 😺

@ctubbsii ctubbsii merged commit 19cbf45 into apache:main May 12, 2021
@ctubbsii ctubbsii added this to the 2.1.0 milestone Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug in VolumeManager.FileType parsing of volume

3 participants