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 2022/10/25 18:17:18 UTC

[GitHub] [accumulo] EdColeman opened a new pull request, #3043: fix upgrade utility - resolves #3041

EdColeman opened a new pull request, #3043:
URL: https://github.com/apache/accumulo/pull/3043

   Fixes #3041 by checking  for upgraded props during the upgrade sequence at cluster start when the properties were previously converted with the config-upgrade utility
   
   - includes minor check-style suggestion to factor common code in if statement path.


-- 
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] EdColeman commented on pull request #3043: fix upgrade utility - resolves #3041

Posted by GitBox <gi...@apache.org>.
EdColeman commented on PR #3043:
URL: https://github.com/apache/accumulo/pull/3043#issuecomment-1291084758

   A full set of ITs passed - FateIT needed to be rerun.
   


-- 
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] EdColeman merged pull request #3043: fix upgrade utility - resolves #3041

Posted by GitBox <gi...@apache.org>.
EdColeman merged PR #3043:
URL: https://github.com/apache/accumulo/pull/3043


-- 
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 #3043: fix upgrade utility - resolves #3041

Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on PR #3043:
URL: https://github.com/apache/accumulo/pull/3043#issuecomment-1291020440

   I was able to reproduce the bug in #3041 with the steps given on that ticket. Then tried the same steps again except substituted this branch and everything seemed to work fine.


-- 
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