You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "DomGarguilo (via GitHub)" <gi...@apache.org> on 2023/03/30 16:55:37 UTC
[GitHub] [accumulo] DomGarguilo opened a new pull request, #3269: Add test for new Validator
DomGarguilo opened a new pull request, #3269:
URL: https://github.com/apache/accumulo/pull/3269
This PR adds a test for the new `NOT_METADATA_TABLE_ID` `Validator`
--
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 unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] DomGarguilo merged pull request #3269: Add test for new Validator
Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo merged PR #3269:
URL: https://github.com/apache/accumulo/pull/3269
--
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 unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] DomGarguilo commented on pull request #3269: Add test for new Validator
Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on PR #3269:
URL: https://github.com/apache/accumulo/pull/3269#issuecomment-1492475160
The new `NOT_METADATA_TABLE_ID` `Validator` was added as part of 3250.
This PR is targeting the `elasticity` branch because it contains the changes made in 3250 which include the new `Validator` where main does not yet have these changes nor the new `Validator`.
--
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 unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on pull request #3269: Add test for new Validator
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #3269:
URL: https://github.com/apache/accumulo/pull/3269#issuecomment-1492464010
Can you provide a reference for where the new Validator was added? It would help to know when / why it was added in order to understand whether the validator is supposed to be checking "not any metadata table (neither accumulo.root nor accumulo.metadata, nor any other future metadata table)" or "not the specific metadata table named accumulo.metadata"
--
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 unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org