You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2020/09/12 02:38:37 UTC
[GitHub] [incubator-dolphinscheduler] yangyichao-mango opened a new issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
yangyichao-mango opened a new issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725
**Describe the question**
Change the no to taskinstance, processinstance, processdefinition id
**What are the current deficiencies and the benefits of improvement**
- It can help user to locate the params of taskinstance, processinstance, processdefinition faster.
**Which version of DolphinScheduler:**
-[dev]
**Describe alternatives you've considered**
----------------------------------------------------------------
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] [incubator-dolphinscheduler] yangyichao-mango commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
yangyichao-mango commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-693170677
Easy to implement， anyone who are interested in this feature, welcome to leave a message.
----------------------------------------------------------------
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] [incubator-dolphinscheduler] kalencaya commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
kalencaya commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-752044385
I'm confused what the issue means.
I just change the auto-generated number beside the checkbox to the askinstance、processinstance、processdefinition id, but refused by pr reviewer.
can point out what ui style this issue want to achive?
the follow image provided by pr reviewer.
----------------------------------------------------------------
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] [incubator-dolphinscheduler] CalvinKirs commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
CalvinKirs commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-745173456
> > > hello, I am interested in this feature, and I want to contribute, can you assign it to me?
> >
> >
> > Very welcome, if you encounter any problems, you can also communicate with me
>
> Do I need to write any code?
Do you understand this question?
----------------------------------------------------------------
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] [incubator-dolphinscheduler] dailidong closed issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
dailidong closed issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725
----------------------------------------------------------------
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] [incubator-dolphinscheduler] CalvinKirs commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
CalvinKirs commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-745181414
> > > > > hello, I am interested in this feature, and I want to contribute, can you assign it to me?
> > > >
> > > >
> > > > Very welcome, if you encounter any problems, you can also communicate with me
> > >
> > >
> > > Do I need to write any code?
> >
> >
> > Do you understand this question?
>
> well, my mistake...It seems that I can not address the issue, please forgive me😥😥you can assign this issue to others.
ok,If there is a chance in the future, I look forward to your contribution.
----------------------------------------------------------------
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] [incubator-dolphinscheduler] kalencaya edited a comment on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
kalencaya edited a comment on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-752044385
I'm confused what the issue means.
I just change the auto-generated number beside the checkbox to the taskinstance、processinstance、processdefinition id, but refused by pr reviewer.
can point out what ui style this issue want to achive?
the follow image provided by pr reviewer, it is also what i fixed.
----------------------------------------------------------------
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] [incubator-dolphinscheduler] shiyi23 commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
shiyi23 commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-745165325
hello, I am interested in this feature, and I want to contribute, can you assign it to me?
----------------------------------------------------------------
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] [incubator-dolphinscheduler] shiyi23 commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
shiyi23 commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-745172212
> > hello, I am interested in this feature, and I want to contribute, can you assign it to me?
>
> Very welcome, if you encounter any problems, you can also communicate with me
Do I need to write any code?
----------------------------------------------------------------
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] [incubator-dolphinscheduler] CalvinKirs commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
CalvinKirs commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-750895150
> hi, can assign it to me? I'm using dolphinscheduler and want to contribute.
Very welcome, if you encounter any problems, you can also communicate with me
----------------------------------------------------------------
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] [incubator-dolphinscheduler] kalencaya edited a comment on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
kalencaya edited a comment on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-752044385
I'm confused what the issue means.
I just change the auto-generated number beside the checkbox to the askinstance、processinstance、processdefinition id, but refused by pr reviewer.
can point out what ui style this issue want to achive?
the follow image provided by pr reviewer, it is also what i fixed.
----------------------------------------------------------------
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] [incubator-dolphinscheduler] shiyi23 commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
shiyi23 commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-745178805
> > > > hello, I am interested in this feature, and I want to contribute, can you assign it to me?
> > >
> > >
> > > Very welcome, if you encounter any problems, you can also communicate with me
> >
> >
> > Do I need to write any code?
>
> Do you understand this question?
well, my mistake...It seems that I can not address the issue, please forgive me😥😥you can assign this issue to others.
----------------------------------------------------------------
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] [incubator-dolphinscheduler] kalencaya commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
kalencaya commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-750873492
hi, can assign it to me? I'm using dolphinscheduler and want to contribute.
----------------------------------------------------------------
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] [incubator-dolphinscheduler] dailidong commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
dailidong commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-770224270
> I'm confused what the issue means.
> I just change the auto-generated number beside the checkbox to the taskinstance、processinstance、processdefinition id, but refused by pr reviewer.
> can point out what ui style this issue want to achive?
> the follow image provided by pr reviewer, it is also what i fixed.
> 
sorry for reply late, I re-check this issue, I think the issue is right, I will talk about the PR reviewer
----------------------------------------------------------------
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] [incubator-dolphinscheduler] shiyi23 commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
shiyi23 commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-745182249
> > > > > > hello, I am interested in this feature, and I want to contribute, can you assign it to me?
> > > > >
> > > > >
> > > > > Very welcome, if you encounter any problems, you can also communicate with me
> > > >
> > > >
> > > > Do I need to write any code?
> > >
> > >
> > > Do you understand this question?
> >
> >
> > well, my mistake...It seems that I can not address the issue, please forgive me😥😥you can assign this issue to others.
>
> ok,If there is a chance in the future, I look forward to your contribution.
Thanks your encouragement！
----------------------------------------------------------------
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] [incubator-dolphinscheduler] CalvinKirs commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
CalvinKirs commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-745168610
> hello, I am interested in this feature, and I want to contribute, can you assign it to me?
Very welcome, if you encounter any problems, you can also communicate with me
----------------------------------------------------------------
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] [incubator-dolphinscheduler] dailidong closed issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
dailidong closed issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725
----------------------------------------------------------------
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] [incubator-dolphinscheduler] zhuangchong commented on issue #3725: [Improvement][ui] Change the no to taskinstance, processinstance, processdefinition id
Posted by GitBox <gi...@apache.org>.
zhuangchong commented on issue #3725:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/3725#issuecomment-692411812
+1
----------------------------------------------------------------
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