Skip to content

Adds text description to clarify id is namespace or a table id in the string passed to CheckConfigUtil#3158

Merged
EdColeman merged 1 commit into
apache:mainfrom
EdColeman:config_check_id2
Jan 10, 2023
Merged

Adds text description to clarify id is namespace or a table id in the string passed to CheckConfigUtil#3158
EdColeman merged 1 commit into
apache:mainfrom
EdColeman:config_check_id2

Conversation

@EdColeman
Copy link
Copy Markdown
Contributor

Suggested in comment on #3141 after merge.

@EdColeman
Copy link
Copy Markdown
Contributor Author

The log messages generated with this change now look like:

2023-01-09T18:54:00,891 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.name) for namespace id: +accumulo
2023-01-09T18:54:00,891 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.peer.password.peer1) for namespace id: +accumulo
2023-01-09T18:54:00,891 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.peer.peer1) for namespace id: +accumulo
2023-01-09T18:54:00,891 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.peer.user.peer1) for namespace id: +accumulo
2023-01-09T18:54:00,900 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.name) for table id: +r
2023-01-09T18:54:00,900 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.peer.password.peer1) for table id: +r
2023-01-09T18:54:00,901 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.peer.peer1) for table id: +r
2023-01-09T18:54:00,901 [conf.ConfigCheckUtil] WARN : BAD CONFIG unrecognized property key (replication.peer.user.peer1) for table id: +r


@EdColeman EdColeman merged commit 7e6df2f into apache:main Jan 10, 2023
@EdColeman EdColeman deleted the config_check_id2 branch January 10, 2023 16:03
@ctubbsii ctubbsii added this to the 3.0.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.

4 participants