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/12/01 18:16:01 UTC
[GitHub] [accumulo] dlmarion opened a new pull request, #3102: Revert "Allow ScanServers to scan offline tables (#3082)" (#3101)
dlmarion opened a new pull request, #3102:
URL: https://github.com/apache/accumulo/pull/3102
This reverts commit 557125539cf35362c69002ad72b00af5dd69e672.
--
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 #3102: Revert "Allow ScanServers to scan offline tables (#3082)" (#3101)
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on PR #3102:
URL: https://github.com/apache/accumulo/pull/3102#issuecomment-1334482105
To fix the 2.1 branch, I'm going to force bush it back to 0928f740f9, which is where 2.1 should have been at. I'm not sure why or when commit b783a1f got added in. It looks like perhaps that was done on the command line.
--
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 #3102: Revert "Allow ScanServers to scan offline tables (#3082)" (#3101)
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on PR #3102:
URL: https://github.com/apache/accumulo/pull/3102#issuecomment-1334477286
It looks like this was squashed when this was commited to the main branch... it should have been a merge commit, to bring the history of the 2.1 branch (specifically at 0928f74) into the main branch. However, it looks like something was messed up, and commit b793a1f was added to the 2.1 branch first, which never should have happened. That merged everything in the main branch into the 2.1 branch... which is not what we want.
--
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] dlmarion merged pull request #3102: Revert "Allow ScanServers to scan offline tables (#3082)" (#3101)
Posted by GitBox <gi...@apache.org>.
dlmarion merged PR #3102:
URL: https://github.com/apache/accumulo/pull/3102
--
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 #3102: Revert "Allow ScanServers to scan offline tables (#3082)" (#3101)
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on PR #3102:
URL: https://github.com/apache/accumulo/pull/3102#issuecomment-1334500373
This commit reverts the code in the main branch that was already reverted in 2.1 in #3101. To correctly merge the 2.1 history into the main branch, I've created a noop merge commit 9a787f12b62e865267fb5d28848432048ad3af96 that merges the revert commit in the 2.1 branch at commit 0928f740f9ddf9aeb104d8357f219457e67de925, which was the head of 2.1 at the time.
--
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