You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/06/07 22:46:34 UTC
[GitHub] [accumulo] Manno15 opened a new issue #2149: Properties from 'config' command return unordered
Manno15 opened a new issue #2149:
URL: https://github.com/apache/accumulo/issues/2149
**Describe the bug**
The `config` command inside the accumulo shell returns the properties unordered.
**Versions (OS, Maven, Java, and others, as appropriate):**
- Affected version(s) of this project: 2.1.0-SNAPSHOT. Appears to be from commit: 4ea9bf1ac916d66896eaaba329da04714a1e8cba
**To Reproduce**
Steps to reproduce the behavior (or a link to an example repository that reproduces the problem):
1. With the commit mentioned above, start-up an accumulo instance.
2. Go into the accumulo shell
3. Run the `config` command
**Expected behavior**
The properties should be returned ordered alphabetically.
**Screenshots**
**Additional context**
Each new instance has a different order of properties.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on issue #2149: Properties from 'config' command return unordered
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #2149:
URL: https://github.com/apache/accumulo/issues/2149#issuecomment-856331126
It looks like this is from the use of `Map.copyOf()` to replace populating a `TreeMap` in some places. In some places, this probably doesn't matter, but for the shell it would be nice. Unfortunately, there's no `SortedMap.copyOf()` or anything like that. The easiest fix is probably to use Guava's `ImmutableSortedMap.copyOf()` instead. There might be a few other places where TreeMap was replaced as well.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] foster33 commented on issue #2149: Properties from 'config' command return unordered
Posted by GitBox <gi...@apache.org>.
foster33 commented on issue #2149:
URL: https://github.com/apache/accumulo/issues/2149#issuecomment-856720201
I would like to work on this if no one else is.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] Manno15 commented on issue #2149: Properties from 'config' command return unordered
Posted by GitBox <gi...@apache.org>.
Manno15 commented on issue #2149:
URL: https://github.com/apache/accumulo/issues/2149#issuecomment-856334542
Yep, that appears to be correct. A quick test with `ImmutableSortedMap.copyOf()` worked.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii closed issue #2149: Properties from 'config' command return unordered
Posted by GitBox <gi...@apache.org>.
ctubbsii closed issue #2149:
URL: https://github.com/apache/accumulo/issues/2149
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org