You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@uniffle.apache.org by "smallzhongfeng (via GitHub)" <gi...@apache.org> on 2023/03/27 09:24:41 UTC
[GitHub] [incubator-uniffle] smallzhongfeng opened a new issue, #768: [Subtask] Cli method for blacklist update
smallzhongfeng opened a new issue, #768:
URL: https://github.com/apache/incubator-uniffle/issues/768
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
### Search before asking
- [X] I have searched in the [issues](https://github.com/apache/incubator-uniffle/issues) and found no similar issues.
### Describe the subtask
Currently, updates to the blacklist need to rely on the scheduling thread pool for updates, which is relatively inefficient. We can emulate the Hadoop method of updating the config and add clis for updates, which is more efficient.
### Parent issue
#758
### Are you willing to submit PR?
- [ ] Yes I am willing to submit a PR!
--
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: dev-unsubscribe@uniffle.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] Kwafoor commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "Kwafoor (via GitHub)" <gi...@apache.org>.
Kwafoor commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1540005309
> cc @Kwafoor #770 has been finished. You can go ahead.
OK，I will finish as soon as possible.
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] jerqi closed issue #768: [Subtask] `Cli` method for blacklist update
Posted by "jerqi (via GitHub)" <gi...@apache.org>.
jerqi closed issue #768: [Subtask] `Cli` method for blacklist update
URL: https://github.com/apache/incubator-uniffle/issues/768
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] jerqi commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "jerqi (via GitHub)" <gi...@apache.org>.
jerqi commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1600731842
closed by #931
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] jerqi commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "jerqi (via GitHub)" <gi...@apache.org>.
jerqi commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1565207066
> > cc @Kwafoor #770 has been finished. You can go ahead.
>
> Almost done, just a little more checking and final touches left. I'll submit it in a few days.
We usually use rest api to control admin service. If we need add api, we would better add rest api. Could we keep consistent with other operation. You can refer to https://github.com/apache/incubator-uniffle/pull/684
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] Kwafoor commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "Kwafoor (via GitHub)" <gi...@apache.org>.
Kwafoor commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1501734549
I'm Interested in this, could you assign it to me？Should I wait for [770](https://github.com/apache/incubator-uniffle/issues/770) to be completed before starting?
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] jerqi commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "jerqi (via GitHub)" <gi...@apache.org>.
jerqi commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1525481449
cc @Kwafoor #770 has been finished. You can go ahead.
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] Kwafoor commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "Kwafoor (via GitHub)" <gi...@apache.org>.
Kwafoor commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1563925226
> cc @Kwafoor #770 has been finished. You can go ahead.
Almost done, just a little more checking and final touches left. I'll submit it in a few days.
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] smallzhongfeng commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "smallzhongfeng (via GitHub)" <gi...@apache.org>.
smallzhongfeng commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1502780449
> I'm Interested in this, could you assign it to me？Should I wait for [770](https://github.com/apache/incubator-uniffle/issues/770) to be completed before starting?
Yes, try to wait for #770 to complete.
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] smallzhongfeng commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "smallzhongfeng (via GitHub)" <gi...@apache.org>.
smallzhongfeng commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1502776408
Of course, I assigned it to you.
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] Kwafoor commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "Kwafoor (via GitHub)" <gi...@apache.org>.
Kwafoor commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1566363992
> > > cc @Kwafoor #770 has been finished. You can go ahead.
> >
> >
> > Almost done, just a little more checking and final touches left. I'll submit it in a few days.
>
> @Kwafoor We usually use rest api to control admin service. If we need add api, we would better add rest api. Could we keep consistent with other operation. You can refer to #684
I have implemented control admin service with reference to yarn through grpc, but I will refer to https://github.com/apache/incubator-uniffle/pull/684 to see how to achieve it through rest.
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-uniffle] Kwafoor commented on issue #768: [Subtask] `Cli` method for blacklist update
Posted by "Kwafoor (via GitHub)" <gi...@apache.org>.
Kwafoor commented on issue #768:
URL: https://github.com/apache/incubator-uniffle/issues/768#issuecomment-1566363728
> I have implemented control admin service with reference to yarn through grpc, but I will refer to #684 to see how to achieve it through rest.
--
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: dev-unsubscribe@uniffle.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org