You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@fluo.apache.org by GitBox <gi...@apache.org> on 2021/06/04 03:41:27 UTC
[GitHub] [fluo-muchos] keith-turner opened a new pull request #396: Update accumulo-env.sh for external compactions
keith-turner opened a new pull request #396:
URL: https://github.com/apache/fluo-muchos/pull/396
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] ctubbsii commented on pull request #396: Update accumulo-env.sh for external compactions
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #396:
URL: https://github.com/apache/fluo-muchos/pull/396#issuecomment-856335261
> I used the Github UI to make the change and it does that.
Yeah, I've been annoyed by that too. If you don't have write access, and try to edit a file using GitHub's built-in text editor, it creates a branch in your fork. If you *do* have write access, it doesn't ask, but just creates the branch in the upstream repo. Very annoying. It should at least prompt which fork you want to create the branch in.
> Thanks for deleting the branch
No problem :smiley_cat:
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] keith-turner commented on pull request #396: Update accumulo-env.sh for external compactions
Posted by GitBox <gi...@apache.org>.
keith-turner commented on pull request #396:
URL: https://github.com/apache/fluo-muchos/pull/396#issuecomment-856321780
> I was curious if there was any reason why you pushed your branch to this remote directly?
@arvindshmicrosoft I used the Github UI to make the change and it does that. I wish it would create the branch in my fork, but it does not. I was in a bit of a hurry when I made the change, I don't use GH web ui often for changes because it create the branch. Thanks for deleting the branch @ctubbsii, I should have done that when I merged.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] keith-turner merged pull request #396: Update accumulo-env.sh for external compactions
Posted by GitBox <gi...@apache.org>.
keith-turner merged pull request #396:
URL: https://github.com/apache/fluo-muchos/pull/396
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] arvindshmicrosoft commented on pull request #396: Update accumulo-env.sh for external compactions
Posted by GitBox <gi...@apache.org>.
arvindshmicrosoft commented on pull request #396:
URL: https://github.com/apache/fluo-muchos/pull/396#issuecomment-856309591
Low priority question - I noticed that the CI checks ran twice. I figured this is because Keith pushed his branch to this repo - which caused the `push` trigger for the CI to activate; in addition to that, the regular `pr` trigger activated as well. @keith-turner, I was curious if there was any reason why you pushed your branch to this remote directly?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] keith-turner merged pull request #396: Update accumulo-env.sh for external compactions
Posted by GitBox <gi...@apache.org>.
keith-turner merged pull request #396:
URL: https://github.com/apache/fluo-muchos/pull/396
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org