You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "keith-turner (via GitHub)" <gi...@apache.org> on 2025/09/22 19:07:38 UTC
[PR] fixes tests that place table props in site config [accumulo]
keith-turner opened a new pull request, #5912:
URL: https://github.com/apache/accumulo/pull/5912
These are some of the easier fixes to test that were broken by the changes in #5886. Removed setting table props in the site config in the test.
--
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
Re: [PR] fixes tests that place table props in site config [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner merged PR #5912:
URL: https://github.com/apache/accumulo/pull/5912
--
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
Re: [PR] fixes tests that place table props in site config [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on PR #5912:
URL: https://github.com/apache/accumulo/pull/5912#issuecomment-3325710150
Pushed f648af7b26e68f3617ef266093c6d847a5356e58 to main w/ a fix to two test that followed the same pattern as a fix in this PR for a GC IT.
--
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
Re: [PR] fixes tests that place table props in site config [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on code in PR #5912:
URL: https://github.com/apache/accumulo/pull/5912#discussion_r2370002025
##########
test/src/main/java/org/apache/accumulo/test/functional/GarbageCollectorTrashEnabledIT.java:
##########
@@ -67,6 +66,7 @@ public void testTrashHadoopEnabledAccumuloEnabled() throws Exception {
final FileSystem fs = super.getCluster().getFileSystem();
super.makeTrashDir(fs);
try (AccumuloClient c = Accumulo.newClient().from(getClientProperties()).build()) {
+ c.namespaceOperations().setProperty("", Property.TABLE_MAJC_RATIO.getKey(), "5.0");
Review Comment:
```suggestion
c.namespaceOperations().setProperty(Namespace.DEFAULT.name(), Property.TABLE_MAJC_RATIO.getKey(), "5.0");
```
--
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