Hello,
I have noticed there is a high verbosity coming from the PR into the JIRA which makes it hard to focus on the message or the content of the PR through the JIRA itself. Has there been a discussion over minimizing the details displayed back on the PR's JIRA? Maybe just add the PR link and the description of the changes rather than the entire template? Any thoughts? Regards, -- Neelesh Srinivas Salian |
Actually that is a good suggestion. I know from other Apache projects
that they only mirror the initial description of the pull request but not the discussion. I agree with you that it's very hard to have meaningful discussion in JIRA if it is overlapped with GitHub comments. Cheers, Max On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian <[hidden email]> wrote: > Hello, > > I have noticed there is a high verbosity coming from the PR into the JIRA > which makes it hard to focus on the message or the content of the PR > through the JIRA itself. > Has there been a discussion over minimizing the details displayed back on > the PR's JIRA? > > Maybe just add the PR link and the description of the changes rather than > the entire template? > > Any thoughts? > > Regards, > > -- > Neelesh Srinivas Salian |
@max - can you contact infra about that?
On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels <[hidden email]> wrote: > Actually that is a good suggestion. I know from other Apache projects > that they only mirror the initial description of the pull request but > not the discussion. I agree with you that it's very hard to have > meaningful discussion in JIRA if it is overlapped with GitHub > comments. > > Cheers, > Max > > On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian <[hidden email]> > wrote: > > Hello, > > > > I have noticed there is a high verbosity coming from the PR into the JIRA > > which makes it hard to focus on the message or the content of the PR > > through the JIRA itself. > > Has there been a discussion over minimizing the details displayed back on > > the PR's JIRA? > > > > Maybe just add the PR link and the description of the changes rather than > > the entire template? > > > > Any thoughts? > > > > Regards, > > > > -- > > Neelesh Srinivas Salian > |
Sure, I've filed a JIRA: https://issues.apache.org/jira/browse/INFRA-12456
On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email]> wrote: > @max - can you contact infra about that? > > On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels <[hidden email]> wrote: > >> Actually that is a good suggestion. I know from other Apache projects >> that they only mirror the initial description of the pull request but >> not the discussion. I agree with you that it's very hard to have >> meaningful discussion in JIRA if it is overlapped with GitHub >> comments. >> >> Cheers, >> Max >> >> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian <[hidden email]> >> wrote: >> > Hello, >> > >> > I have noticed there is a high verbosity coming from the PR into the JIRA >> > which makes it hard to focus on the message or the content of the PR >> > through the JIRA itself. >> > Has there been a discussion over minimizing the details displayed back on >> > the PR's JIRA? >> > >> > Maybe just add the PR link and the description of the changes rather than >> > the entire template? >> > >> > Any thoughts? >> > >> > Regards, >> > >> > -- >> > Neelesh Srinivas Salian >> |
From what I understand so far, the message mirroring can be adjusted
in the follow parts: 1) GitHub PR description 2) GitHub Diff comments 3) GitHub Main thread comments We can choose to put these as 1) The full GitHub comment 2) Only supply a link to the GitHub comment Next, we can store all posts to JIRA 1) In the JIRA main comments 2) In the Work Log I think it would be a nice setup to have the GitHub PR description and comments directly in the JIRA comments. Diff comments should go in the Work Log. On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels <[hidden email]> wrote: > Sure, I've filed a JIRA: https://issues.apache.org/jira/browse/INFRA-12456 > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email]> wrote: >> @max - can you contact infra about that? >> >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels <[hidden email]> wrote: >> >>> Actually that is a good suggestion. I know from other Apache projects >>> that they only mirror the initial description of the pull request but >>> not the discussion. I agree with you that it's very hard to have >>> meaningful discussion in JIRA if it is overlapped with GitHub >>> comments. >>> >>> Cheers, >>> Max >>> >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian <[hidden email]> >>> wrote: >>> > Hello, >>> > >>> > I have noticed there is a high verbosity coming from the PR into the JIRA >>> > which makes it hard to focus on the message or the content of the PR >>> > through the JIRA itself. >>> > Has there been a discussion over minimizing the details displayed back on >>> > the PR's JIRA? >>> > >>> > Maybe just add the PR link and the description of the changes rather than >>> > the entire template? >>> > >>> > Any thoughts? >>> > >>> > Regards, >>> > >>> > -- >>> > Neelesh Srinivas Salian >>> |
+1
I didn't know that we can put the comments into the work log. Will work log updates still lead to notifications on the mailing list? On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels <[hidden email]> wrote: > From what I understand so far, the message mirroring can be adjusted > in the follow parts: > > 1) GitHub PR description > 2) GitHub Diff comments > 3) GitHub Main thread comments > > We can choose to put these as > > 1) The full GitHub comment > 2) Only supply a link to the GitHub comment > > Next, we can store all posts to JIRA > > 1) In the JIRA main comments > 2) In the Work Log > > > I think it would be a nice setup to have the GitHub PR description and > comments directly in the JIRA comments. Diff comments should go in the > Work Log. > > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels <[hidden email]> > wrote: > > Sure, I've filed a JIRA: https://issues.apache.org/ > jira/browse/INFRA-12456 > > > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email]> wrote: > >> @max - can you contact infra about that? > >> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels <[hidden email]> > wrote: > >> > >>> Actually that is a good suggestion. I know from other Apache projects > >>> that they only mirror the initial description of the pull request but > >>> not the discussion. I agree with you that it's very hard to have > >>> meaningful discussion in JIRA if it is overlapped with GitHub > >>> comments. > >>> > >>> Cheers, > >>> Max > >>> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian <[hidden email] > > > >>> wrote: > >>> > Hello, > >>> > > >>> > I have noticed there is a high verbosity coming from the PR into the > JIRA > >>> > which makes it hard to focus on the message or the content of the PR > >>> > through the JIRA itself. > >>> > Has there been a discussion over minimizing the details displayed > back on > >>> > the PR's JIRA? > >>> > > >>> > Maybe just add the PR link and the description of the changes rather > than > >>> > the entire template? > >>> > > >>> > Any thoughts? > >>> > > >>> > Regards, > >>> > > >>> > -- > >>> > Neelesh Srinivas Salian > >>> > |
Thanks Max for coordinating the effort.
+1 as well. On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger <[hidden email]> wrote: > +1 > > I didn't know that we can put the comments into the work log. > Will work log updates still lead to notifications on the mailing list? > > On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels <[hidden email]> > wrote: > > > From what I understand so far, the message mirroring can be adjusted > > in the follow parts: > > > > 1) GitHub PR description > > 2) GitHub Diff comments > > 3) GitHub Main thread comments > > > > We can choose to put these as > > > > 1) The full GitHub comment > > 2) Only supply a link to the GitHub comment > > > > Next, we can store all posts to JIRA > > > > 1) In the JIRA main comments > > 2) In the Work Log > > > > > > I think it would be a nice setup to have the GitHub PR description and > > comments directly in the JIRA comments. Diff comments should go in the > > Work Log. > > > > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels <[hidden email]> > > wrote: > > > Sure, I've filed a JIRA: https://issues.apache.org/ > > jira/browse/INFRA-12456 > > > > > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email]> > wrote: > > >> @max - can you contact infra about that? > > >> > > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels <[hidden email]> > > wrote: > > >> > > >>> Actually that is a good suggestion. I know from other Apache projects > > >>> that they only mirror the initial description of the pull request but > > >>> not the discussion. I agree with you that it's very hard to have > > >>> meaningful discussion in JIRA if it is overlapped with GitHub > > >>> comments. > > >>> > > >>> Cheers, > > >>> Max > > >>> > > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < > [hidden email] > > > > > >>> wrote: > > >>> > Hello, > > >>> > > > >>> > I have noticed there is a high verbosity coming from the PR into > the > > JIRA > > >>> > which makes it hard to focus on the message or the content of the > PR > > >>> > through the JIRA itself. > > >>> > Has there been a discussion over minimizing the details displayed > > back on > > >>> > the PR's JIRA? > > >>> > > > >>> > Maybe just add the PR link and the description of the changes > rather > > than > > >>> > the entire template? > > >>> > > > >>> > Any thoughts? > > >>> > > > >>> > Regards, > > >>> > > > >>> > -- > > >>> > Neelesh Srinivas Salian > > >>> > > > -- Neelesh Srinivas Salian Customer Operations Engineer |
> Will work log updates still lead to notifications on the mailing list?
I don't know but it is most likely configurable. I would prefer if we didn't get notifications for work log items because these would again be duplicates of the GitHub notifications. On Mon, Aug 29, 2016 at 5:39 PM, Neelesh Salian <[hidden email]> wrote: > Thanks Max for coordinating the effort. > +1 as well. > > > On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger <[hidden email]> wrote: > >> +1 >> >> I didn't know that we can put the comments into the work log. >> Will work log updates still lead to notifications on the mailing list? >> >> On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels <[hidden email]> >> wrote: >> >> > From what I understand so far, the message mirroring can be adjusted >> > in the follow parts: >> > >> > 1) GitHub PR description >> > 2) GitHub Diff comments >> > 3) GitHub Main thread comments >> > >> > We can choose to put these as >> > >> > 1) The full GitHub comment >> > 2) Only supply a link to the GitHub comment >> > >> > Next, we can store all posts to JIRA >> > >> > 1) In the JIRA main comments >> > 2) In the Work Log >> > >> > >> > I think it would be a nice setup to have the GitHub PR description and >> > comments directly in the JIRA comments. Diff comments should go in the >> > Work Log. >> > >> > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels <[hidden email]> >> > wrote: >> > > Sure, I've filed a JIRA: https://issues.apache.org/ >> > jira/browse/INFRA-12456 >> > > >> > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email]> >> wrote: >> > >> @max - can you contact infra about that? >> > >> >> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels <[hidden email]> >> > wrote: >> > >> >> > >>> Actually that is a good suggestion. I know from other Apache projects >> > >>> that they only mirror the initial description of the pull request but >> > >>> not the discussion. I agree with you that it's very hard to have >> > >>> meaningful discussion in JIRA if it is overlapped with GitHub >> > >>> comments. >> > >>> >> > >>> Cheers, >> > >>> Max >> > >>> >> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < >> [hidden email] >> > > >> > >>> wrote: >> > >>> > Hello, >> > >>> > >> > >>> > I have noticed there is a high verbosity coming from the PR into >> the >> > JIRA >> > >>> > which makes it hard to focus on the message or the content of the >> PR >> > >>> > through the JIRA itself. >> > >>> > Has there been a discussion over minimizing the details displayed >> > back on >> > >>> > the PR's JIRA? >> > >>> > >> > >>> > Maybe just add the PR link and the description of the changes >> rather >> > than >> > >>> > the entire template? >> > >>> > >> > >>> > Any thoughts? >> > >>> > >> > >>> > Regards, >> > >>> > >> > >>> > -- >> > >>> > Neelesh Srinivas Salian >> > >>> >> > >> > > > > -- > Neelesh Srinivas Salian > Customer Operations Engineer |
If there are no objections, I will contact Infra to change the GitHub
JIRA notifications as follows: Jira comments section - initial PR description - comments of the main GitHub thread Jira Work Log - all diff comments On Mon, Aug 29, 2016 at 6:58 PM, Maximilian Michels <[hidden email]> wrote: >> Will work log updates still lead to notifications on the mailing list? > > I don't know but it is most likely configurable. I would prefer if we > didn't get notifications for work log items because these would again > be duplicates of the GitHub notifications. > > On Mon, Aug 29, 2016 at 5:39 PM, Neelesh Salian <[hidden email]> wrote: >> Thanks Max for coordinating the effort. >> +1 as well. >> >> >> On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger <[hidden email]> wrote: >> >>> +1 >>> >>> I didn't know that we can put the comments into the work log. >>> Will work log updates still lead to notifications on the mailing list? >>> >>> On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels <[hidden email]> >>> wrote: >>> >>> > From what I understand so far, the message mirroring can be adjusted >>> > in the follow parts: >>> > >>> > 1) GitHub PR description >>> > 2) GitHub Diff comments >>> > 3) GitHub Main thread comments >>> > >>> > We can choose to put these as >>> > >>> > 1) The full GitHub comment >>> > 2) Only supply a link to the GitHub comment >>> > >>> > Next, we can store all posts to JIRA >>> > >>> > 1) In the JIRA main comments >>> > 2) In the Work Log >>> > >>> > >>> > I think it would be a nice setup to have the GitHub PR description and >>> > comments directly in the JIRA comments. Diff comments should go in the >>> > Work Log. >>> > >>> > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels <[hidden email]> >>> > wrote: >>> > > Sure, I've filed a JIRA: https://issues.apache.org/ >>> > jira/browse/INFRA-12456 >>> > > >>> > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email]> >>> wrote: >>> > >> @max - can you contact infra about that? >>> > >> >>> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels <[hidden email]> >>> > wrote: >>> > >> >>> > >>> Actually that is a good suggestion. I know from other Apache projects >>> > >>> that they only mirror the initial description of the pull request but >>> > >>> not the discussion. I agree with you that it's very hard to have >>> > >>> meaningful discussion in JIRA if it is overlapped with GitHub >>> > >>> comments. >>> > >>> >>> > >>> Cheers, >>> > >>> Max >>> > >>> >>> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < >>> [hidden email] >>> > > >>> > >>> wrote: >>> > >>> > Hello, >>> > >>> > >>> > >>> > I have noticed there is a high verbosity coming from the PR into >>> the >>> > JIRA >>> > >>> > which makes it hard to focus on the message or the content of the >>> PR >>> > >>> > through the JIRA itself. >>> > >>> > Has there been a discussion over minimizing the details displayed >>> > back on >>> > >>> > the PR's JIRA? >>> > >>> > >>> > >>> > Maybe just add the PR link and the description of the changes >>> rather >>> > than >>> > >>> > the entire template? >>> > >>> > >>> > >>> > Any thoughts? >>> > >>> > >>> > >>> > Regards, >>> > >>> > >>> > >>> > -- >>> > >>> > Neelesh Srinivas Salian >>> > >>> >>> > >>> >> >> >> >> -- >> Neelesh Srinivas Salian >> Customer Operations Engineer |
Sounds good to me!
On Fri, Sep 2, 2016 at 3:08 PM, Maximilian Michels <[hidden email]> wrote: > If there are no objections, I will contact Infra to change the GitHub > JIRA notifications as follows: > > Jira comments section > - initial PR description > - comments of the main GitHub thread > > Jira Work Log > - all diff comments > > > On Mon, Aug 29, 2016 at 6:58 PM, Maximilian Michels <[hidden email]> > wrote: > >> Will work log updates still lead to notifications on the mailing list? > > > > I don't know but it is most likely configurable. I would prefer if we > > didn't get notifications for work log items because these would again > > be duplicates of the GitHub notifications. > > > > On Mon, Aug 29, 2016 at 5:39 PM, Neelesh Salian <[hidden email]> > wrote: > >> Thanks Max for coordinating the effort. > >> +1 as well. > >> > >> > >> On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger <[hidden email]> > wrote: > >> > >>> +1 > >>> > >>> I didn't know that we can put the comments into the work log. > >>> Will work log updates still lead to notifications on the mailing list? > >>> > >>> On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels <[hidden email]> > >>> wrote: > >>> > >>> > From what I understand so far, the message mirroring can be adjusted > >>> > in the follow parts: > >>> > > >>> > 1) GitHub PR description > >>> > 2) GitHub Diff comments > >>> > 3) GitHub Main thread comments > >>> > > >>> > We can choose to put these as > >>> > > >>> > 1) The full GitHub comment > >>> > 2) Only supply a link to the GitHub comment > >>> > > >>> > Next, we can store all posts to JIRA > >>> > > >>> > 1) In the JIRA main comments > >>> > 2) In the Work Log > >>> > > >>> > > >>> > I think it would be a nice setup to have the GitHub PR description > and > >>> > comments directly in the JIRA comments. Diff comments should go in > the > >>> > Work Log. > >>> > > >>> > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels <[hidden email]> > >>> > wrote: > >>> > > Sure, I've filed a JIRA: https://issues.apache.org/ > >>> > jira/browse/INFRA-12456 > >>> > > > >>> > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email]> > >>> wrote: > >>> > >> @max - can you contact infra about that? > >>> > >> > >>> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels < > [hidden email]> > >>> > wrote: > >>> > >> > >>> > >>> Actually that is a good suggestion. I know from other Apache > projects > >>> > >>> that they only mirror the initial description of the pull > request but > >>> > >>> not the discussion. I agree with you that it's very hard to have > >>> > >>> meaningful discussion in JIRA if it is overlapped with GitHub > >>> > >>> comments. > >>> > >>> > >>> > >>> Cheers, > >>> > >>> Max > >>> > >>> > >>> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < > >>> [hidden email] > >>> > > > >>> > >>> wrote: > >>> > >>> > Hello, > >>> > >>> > > >>> > >>> > I have noticed there is a high verbosity coming from the PR > into > >>> the > >>> > JIRA > >>> > >>> > which makes it hard to focus on the message or the content of > the > >>> PR > >>> > >>> > through the JIRA itself. > >>> > >>> > Has there been a discussion over minimizing the details > displayed > >>> > back on > >>> > >>> > the PR's JIRA? > >>> > >>> > > >>> > >>> > Maybe just add the PR link and the description of the changes > >>> rather > >>> > than > >>> > >>> > the entire template? > >>> > >>> > > >>> > >>> > Any thoughts? > >>> > >>> > > >>> > >>> > Regards, > >>> > >>> > > >>> > >>> > -- > >>> > >>> > Neelesh Srinivas Salian > >>> > >>> > >>> > > >>> > >> > >> > >> > >> -- > >> Neelesh Srinivas Salian > >> Customer Operations Engineer > |
+1
Thanks Max! 2016-09-02 15:20 GMT+02:00 Stephan Ewen <[hidden email]>: > Sounds good to me! > > On Fri, Sep 2, 2016 at 3:08 PM, Maximilian Michels <[hidden email]> wrote: > > > If there are no objections, I will contact Infra to change the GitHub > > JIRA notifications as follows: > > > > Jira comments section > > - initial PR description > > - comments of the main GitHub thread > > > > Jira Work Log > > - all diff comments > > > > > > On Mon, Aug 29, 2016 at 6:58 PM, Maximilian Michels <[hidden email]> > > wrote: > > >> Will work log updates still lead to notifications on the mailing list? > > > > > > I don't know but it is most likely configurable. I would prefer if we > > > didn't get notifications for work log items because these would again > > > be duplicates of the GitHub notifications. > > > > > > On Mon, Aug 29, 2016 at 5:39 PM, Neelesh Salian <[hidden email]> > > wrote: > > >> Thanks Max for coordinating the effort. > > >> +1 as well. > > >> > > >> > > >> On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger <[hidden email]> > > wrote: > > >> > > >>> +1 > > >>> > > >>> I didn't know that we can put the comments into the work log. > > >>> Will work log updates still lead to notifications on the mailing > list? > > >>> > > >>> On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels <[hidden email] > > > > >>> wrote: > > >>> > > >>> > From what I understand so far, the message mirroring can be > adjusted > > >>> > in the follow parts: > > >>> > > > >>> > 1) GitHub PR description > > >>> > 2) GitHub Diff comments > > >>> > 3) GitHub Main thread comments > > >>> > > > >>> > We can choose to put these as > > >>> > > > >>> > 1) The full GitHub comment > > >>> > 2) Only supply a link to the GitHub comment > > >>> > > > >>> > Next, we can store all posts to JIRA > > >>> > > > >>> > 1) In the JIRA main comments > > >>> > 2) In the Work Log > > >>> > > > >>> > > > >>> > I think it would be a nice setup to have the GitHub PR description > > and > > >>> > comments directly in the JIRA comments. Diff comments should go in > > the > > >>> > Work Log. > > >>> > > > >>> > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels < > [hidden email]> > > >>> > wrote: > > >>> > > Sure, I've filed a JIRA: https://issues.apache.org/ > > >>> > jira/browse/INFRA-12456 > > >>> > > > > >>> > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen <[hidden email] > > > > >>> wrote: > > >>> > >> @max - can you contact infra about that? > > >>> > >> > > >>> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels < > > [hidden email]> > > >>> > wrote: > > >>> > >> > > >>> > >>> Actually that is a good suggestion. I know from other Apache > > projects > > >>> > >>> that they only mirror the initial description of the pull > > request but > > >>> > >>> not the discussion. I agree with you that it's very hard to > have > > >>> > >>> meaningful discussion in JIRA if it is overlapped with GitHub > > >>> > >>> comments. > > >>> > >>> > > >>> > >>> Cheers, > > >>> > >>> Max > > >>> > >>> > > >>> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < > > >>> [hidden email] > > >>> > > > > >>> > >>> wrote: > > >>> > >>> > Hello, > > >>> > >>> > > > >>> > >>> > I have noticed there is a high verbosity coming from the PR > > into > > >>> the > > >>> > JIRA > > >>> > >>> > which makes it hard to focus on the message or the content of > > the > > >>> PR > > >>> > >>> > through the JIRA itself. > > >>> > >>> > Has there been a discussion over minimizing the details > > displayed > > >>> > back on > > >>> > >>> > the PR's JIRA? > > >>> > >>> > > > >>> > >>> > Maybe just add the PR link and the description of the changes > > >>> rather > > >>> > than > > >>> > >>> > the entire template? > > >>> > >>> > > > >>> > >>> > Any thoughts? > > >>> > >>> > > > >>> > >>> > Regards, > > >>> > >>> > > > >>> > >>> > -- > > >>> > >>> > Neelesh Srinivas Salian > > >>> > >>> > > >>> > > > >>> > > >> > > >> > > >> > > >> -- > > >> Neelesh Srinivas Salian > > >> Customer Operations Engineer > > > |
+1
On Fri, Sep 2, 2016 at 4:20 PM, Fabian Hueske <[hidden email]> wrote: > +1 > > Thanks Max! > > 2016-09-02 15:20 GMT+02:00 Stephan Ewen <[hidden email]>: > > > Sounds good to me! > > > > On Fri, Sep 2, 2016 at 3:08 PM, Maximilian Michels <[hidden email]> > wrote: > > > > > If there are no objections, I will contact Infra to change the GitHub > > > JIRA notifications as follows: > > > > > > Jira comments section > > > - initial PR description > > > - comments of the main GitHub thread > > > > > > Jira Work Log > > > - all diff comments > > > > > > > > > On Mon, Aug 29, 2016 at 6:58 PM, Maximilian Michels <[hidden email]> > > > wrote: > > > >> Will work log updates still lead to notifications on the mailing > list? > > > > > > > > I don't know but it is most likely configurable. I would prefer if we > > > > didn't get notifications for work log items because these would again > > > > be duplicates of the GitHub notifications. > > > > > > > > On Mon, Aug 29, 2016 at 5:39 PM, Neelesh Salian < > [hidden email]> > > > wrote: > > > >> Thanks Max for coordinating the effort. > > > >> +1 as well. > > > >> > > > >> > > > >> On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger < > [hidden email]> > > > wrote: > > > >> > > > >>> +1 > > > >>> > > > >>> I didn't know that we can put the comments into the work log. > > > >>> Will work log updates still lead to notifications on the mailing > > list? > > > >>> > > > >>> On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels < > [hidden email] > > > > > > >>> wrote: > > > >>> > > > >>> > From what I understand so far, the message mirroring can be > > adjusted > > > >>> > in the follow parts: > > > >>> > > > > >>> > 1) GitHub PR description > > > >>> > 2) GitHub Diff comments > > > >>> > 3) GitHub Main thread comments > > > >>> > > > > >>> > We can choose to put these as > > > >>> > > > > >>> > 1) The full GitHub comment > > > >>> > 2) Only supply a link to the GitHub comment > > > >>> > > > > >>> > Next, we can store all posts to JIRA > > > >>> > > > > >>> > 1) In the JIRA main comments > > > >>> > 2) In the Work Log > > > >>> > > > > >>> > > > > >>> > I think it would be a nice setup to have the GitHub PR > description > > > and > > > >>> > comments directly in the JIRA comments. Diff comments should go > in > > > the > > > >>> > Work Log. > > > >>> > > > > >>> > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels < > > [hidden email]> > > > >>> > wrote: > > > >>> > > Sure, I've filed a JIRA: https://issues.apache.org/ > > > >>> > jira/browse/INFRA-12456 > > > >>> > > > > > >>> > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen < > [hidden email] > > > > > > >>> wrote: > > > >>> > >> @max - can you contact infra about that? > > > >>> > >> > > > >>> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels < > > > [hidden email]> > > > >>> > wrote: > > > >>> > >> > > > >>> > >>> Actually that is a good suggestion. I know from other Apache > > > projects > > > >>> > >>> that they only mirror the initial description of the pull > > > request but > > > >>> > >>> not the discussion. I agree with you that it's very hard to > > have > > > >>> > >>> meaningful discussion in JIRA if it is overlapped with GitHub > > > >>> > >>> comments. > > > >>> > >>> > > > >>> > >>> Cheers, > > > >>> > >>> Max > > > >>> > >>> > > > >>> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < > > > >>> [hidden email] > > > >>> > > > > > >>> > >>> wrote: > > > >>> > >>> > Hello, > > > >>> > >>> > > > > >>> > >>> > I have noticed there is a high verbosity coming from the PR > > > into > > > >>> the > > > >>> > JIRA > > > >>> > >>> > which makes it hard to focus on the message or the content > of > > > the > > > >>> PR > > > >>> > >>> > through the JIRA itself. > > > >>> > >>> > Has there been a discussion over minimizing the details > > > displayed > > > >>> > back on > > > >>> > >>> > the PR's JIRA? > > > >>> > >>> > > > > >>> > >>> > Maybe just add the PR link and the description of the > changes > > > >>> rather > > > >>> > than > > > >>> > >>> > the entire template? > > > >>> > >>> > > > > >>> > >>> > Any thoughts? > > > >>> > >>> > > > > >>> > >>> > Regards, > > > >>> > >>> > > > > >>> > >>> > -- > > > >>> > >>> > Neelesh Srinivas Salian > > > >>> > >>> > > > >>> > > > > >>> > > > >> > > > >> > > > >> > > > >> -- > > > >> Neelesh Srinivas Salian > > > >> Customer Operations Engineer > > > > > > |
Sorry for late reply, but +1 too
This should still provide tracking of discussion from Github PR and reduce the noise at the same time. Thanks for working on it, Max. - Henry On Fri, Sep 2, 2016 at 8:42 AM, Till Rohrmann <[hidden email]> wrote: > +1 > > On Fri, Sep 2, 2016 at 4:20 PM, Fabian Hueske <[hidden email]> wrote: > > > +1 > > > > Thanks Max! > > > > 2016-09-02 15:20 GMT+02:00 Stephan Ewen <[hidden email]>: > > > > > Sounds good to me! > > > > > > On Fri, Sep 2, 2016 at 3:08 PM, Maximilian Michels <[hidden email]> > > wrote: > > > > > > > If there are no objections, I will contact Infra to change the GitHub > > > > JIRA notifications as follows: > > > > > > > > Jira comments section > > > > - initial PR description > > > > - comments of the main GitHub thread > > > > > > > > Jira Work Log > > > > - all diff comments > > > > > > > > > > > > On Mon, Aug 29, 2016 at 6:58 PM, Maximilian Michels <[hidden email]> > > > > wrote: > > > > >> Will work log updates still lead to notifications on the mailing > > list? > > > > > > > > > > I don't know but it is most likely configurable. I would prefer if > we > > > > > didn't get notifications for work log items because these would > again > > > > > be duplicates of the GitHub notifications. > > > > > > > > > > On Mon, Aug 29, 2016 at 5:39 PM, Neelesh Salian < > > [hidden email]> > > > > wrote: > > > > >> Thanks Max for coordinating the effort. > > > > >> +1 as well. > > > > >> > > > > >> > > > > >> On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger < > > [hidden email]> > > > > wrote: > > > > >> > > > > >>> +1 > > > > >>> > > > > >>> I didn't know that we can put the comments into the work log. > > > > >>> Will work log updates still lead to notifications on the mailing > > > list? > > > > >>> > > > > >>> On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels < > > [hidden email] > > > > > > > > >>> wrote: > > > > >>> > > > > >>> > From what I understand so far, the message mirroring can be > > > adjusted > > > > >>> > in the follow parts: > > > > >>> > > > > > >>> > 1) GitHub PR description > > > > >>> > 2) GitHub Diff comments > > > > >>> > 3) GitHub Main thread comments > > > > >>> > > > > > >>> > We can choose to put these as > > > > >>> > > > > > >>> > 1) The full GitHub comment > > > > >>> > 2) Only supply a link to the GitHub comment > > > > >>> > > > > > >>> > Next, we can store all posts to JIRA > > > > >>> > > > > > >>> > 1) In the JIRA main comments > > > > >>> > 2) In the Work Log > > > > >>> > > > > > >>> > > > > > >>> > I think it would be a nice setup to have the GitHub PR > > description > > > > and > > > > >>> > comments directly in the JIRA comments. Diff comments should go > > in > > > > the > > > > >>> > Work Log. > > > > >>> > > > > > >>> > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels < > > > [hidden email]> > > > > >>> > wrote: > > > > >>> > > Sure, I've filed a JIRA: https://issues.apache.org/ > > > > >>> > jira/browse/INFRA-12456 > > > > >>> > > > > > > >>> > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen < > > [hidden email] > > > > > > > > >>> wrote: > > > > >>> > >> @max - can you contact infra about that? > > > > >>> > >> > > > > >>> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels < > > > > [hidden email]> > > > > >>> > wrote: > > > > >>> > >> > > > > >>> > >>> Actually that is a good suggestion. I know from other > Apache > > > > projects > > > > >>> > >>> that they only mirror the initial description of the pull > > > > request but > > > > >>> > >>> not the discussion. I agree with you that it's very hard to > > > have > > > > >>> > >>> meaningful discussion in JIRA if it is overlapped with > GitHub > > > > >>> > >>> comments. > > > > >>> > >>> > > > > >>> > >>> Cheers, > > > > >>> > >>> Max > > > > >>> > >>> > > > > >>> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < > > > > >>> [hidden email] > > > > >>> > > > > > > >>> > >>> wrote: > > > > >>> > >>> > Hello, > > > > >>> > >>> > > > > > >>> > >>> > I have noticed there is a high verbosity coming from the > PR > > > > into > > > > >>> the > > > > >>> > JIRA > > > > >>> > >>> > which makes it hard to focus on the message or the > content > > of > > > > the > > > > >>> PR > > > > >>> > >>> > through the JIRA itself. > > > > >>> > >>> > Has there been a discussion over minimizing the details > > > > displayed > > > > >>> > back on > > > > >>> > >>> > the PR's JIRA? > > > > >>> > >>> > > > > > >>> > >>> > Maybe just add the PR link and the description of the > > changes > > > > >>> rather > > > > >>> > than > > > > >>> > >>> > the entire template? > > > > >>> > >>> > > > > > >>> > >>> > Any thoughts? > > > > >>> > >>> > > > > > >>> > >>> > Regards, > > > > >>> > >>> > > > > > >>> > >>> > -- > > > > >>> > >>> > Neelesh Srinivas Salian > > > > >>> > >>> > > > > >>> > > > > > >>> > > > > >> > > > > >> > > > > >> > > > > >> -- > > > > >> Neelesh Srinivas Salian > > > > >> Customer Operations Engineer > > > > > > > > > > |
To sum up the activity in https://issues.apache.org/jira/browse/INFRA-12456:
Bit of a bummer, we will have to work on the Perl script which publishes the notifications. If I find some time I might do that but volunteers are welcome: https://git-wip-us.apache.org/repos/infra?p=asfgit-admin.git;a=blob_plain;f=cgi-bin/github.cgi;hb=HEAD On Mon, Sep 5, 2016 at 8:42 PM, Henry Saputra <[hidden email]> wrote: > Sorry for late reply, but +1 too > > This should still provide tracking of discussion from Github PR and reduce > the noise at the same time. > > Thanks for working on it, Max. > > - Henry > > On Fri, Sep 2, 2016 at 8:42 AM, Till Rohrmann <[hidden email]> wrote: > >> +1 >> >> On Fri, Sep 2, 2016 at 4:20 PM, Fabian Hueske <[hidden email]> wrote: >> >> > +1 >> > >> > Thanks Max! >> > >> > 2016-09-02 15:20 GMT+02:00 Stephan Ewen <[hidden email]>: >> > >> > > Sounds good to me! >> > > >> > > On Fri, Sep 2, 2016 at 3:08 PM, Maximilian Michels <[hidden email]> >> > wrote: >> > > >> > > > If there are no objections, I will contact Infra to change the GitHub >> > > > JIRA notifications as follows: >> > > > >> > > > Jira comments section >> > > > - initial PR description >> > > > - comments of the main GitHub thread >> > > > >> > > > Jira Work Log >> > > > - all diff comments >> > > > >> > > > >> > > > On Mon, Aug 29, 2016 at 6:58 PM, Maximilian Michels <[hidden email]> >> > > > wrote: >> > > > >> Will work log updates still lead to notifications on the mailing >> > list? >> > > > > >> > > > > I don't know but it is most likely configurable. I would prefer if >> we >> > > > > didn't get notifications for work log items because these would >> again >> > > > > be duplicates of the GitHub notifications. >> > > > > >> > > > > On Mon, Aug 29, 2016 at 5:39 PM, Neelesh Salian < >> > [hidden email]> >> > > > wrote: >> > > > >> Thanks Max for coordinating the effort. >> > > > >> +1 as well. >> > > > >> >> > > > >> >> > > > >> On Mon, Aug 29, 2016 at 6:57 AM, Robert Metzger < >> > [hidden email]> >> > > > wrote: >> > > > >> >> > > > >>> +1 >> > > > >>> >> > > > >>> I didn't know that we can put the comments into the work log. >> > > > >>> Will work log updates still lead to notifications on the mailing >> > > list? >> > > > >>> >> > > > >>> On Mon, Aug 29, 2016 at 11:52 AM, Maximilian Michels < >> > [hidden email] >> > > > >> > > > >>> wrote: >> > > > >>> >> > > > >>> > From what I understand so far, the message mirroring can be >> > > adjusted >> > > > >>> > in the follow parts: >> > > > >>> > >> > > > >>> > 1) GitHub PR description >> > > > >>> > 2) GitHub Diff comments >> > > > >>> > 3) GitHub Main thread comments >> > > > >>> > >> > > > >>> > We can choose to put these as >> > > > >>> > >> > > > >>> > 1) The full GitHub comment >> > > > >>> > 2) Only supply a link to the GitHub comment >> > > > >>> > >> > > > >>> > Next, we can store all posts to JIRA >> > > > >>> > >> > > > >>> > 1) In the JIRA main comments >> > > > >>> > 2) In the Work Log >> > > > >>> > >> > > > >>> > >> > > > >>> > I think it would be a nice setup to have the GitHub PR >> > description >> > > > and >> > > > >>> > comments directly in the JIRA comments. Diff comments should go >> > in >> > > > the >> > > > >>> > Work Log. >> > > > >>> > >> > > > >>> > On Fri, Aug 19, 2016 at 2:56 PM, Maximilian Michels < >> > > [hidden email]> >> > > > >>> > wrote: >> > > > >>> > > Sure, I've filed a JIRA: https://issues.apache.org/ >> > > > >>> > jira/browse/INFRA-12456 >> > > > >>> > > >> > > > >>> > > On Thu, Aug 18, 2016 at 10:57 AM, Stephan Ewen < >> > [hidden email] >> > > > >> > > > >>> wrote: >> > > > >>> > >> @max - can you contact infra about that? >> > > > >>> > >> >> > > > >>> > >> On Thu, Aug 18, 2016 at 10:25 AM, Maximilian Michels < >> > > > [hidden email]> >> > > > >>> > wrote: >> > > > >>> > >> >> > > > >>> > >>> Actually that is a good suggestion. I know from other >> Apache >> > > > projects >> > > > >>> > >>> that they only mirror the initial description of the pull >> > > > request but >> > > > >>> > >>> not the discussion. I agree with you that it's very hard to >> > > have >> > > > >>> > >>> meaningful discussion in JIRA if it is overlapped with >> GitHub >> > > > >>> > >>> comments. >> > > > >>> > >>> >> > > > >>> > >>> Cheers, >> > > > >>> > >>> Max >> > > > >>> > >>> >> > > > >>> > >>> On Wed, Aug 17, 2016 at 10:52 PM, Neelesh Salian < >> > > > >>> [hidden email] >> > > > >>> > > >> > > > >>> > >>> wrote: >> > > > >>> > >>> > Hello, >> > > > >>> > >>> > >> > > > >>> > >>> > I have noticed there is a high verbosity coming from the >> PR >> > > > into >> > > > >>> the >> > > > >>> > JIRA >> > > > >>> > >>> > which makes it hard to focus on the message or the >> content >> > of >> > > > the >> > > > >>> PR >> > > > >>> > >>> > through the JIRA itself. >> > > > >>> > >>> > Has there been a discussion over minimizing the details >> > > > displayed >> > > > >>> > back on >> > > > >>> > >>> > the PR's JIRA? >> > > > >>> > >>> > >> > > > >>> > >>> > Maybe just add the PR link and the description of the >> > changes >> > > > >>> rather >> > > > >>> > than >> > > > >>> > >>> > the entire template? >> > > > >>> > >>> > >> > > > >>> > >>> > Any thoughts? >> > > > >>> > >>> > >> > > > >>> > >>> > Regards, >> > > > >>> > >>> > >> > > > >>> > >>> > -- >> > > > >>> > >>> > Neelesh Srinivas Salian >> > > > >>> > >>> >> > > > >>> > >> > > > >>> >> > > > >> >> > > > >> >> > > > >> >> > > > >> -- >> > > > >> Neelesh Srinivas Salian >> > > > >> Customer Operations Engineer >> > > > >> > > >> > >> |
Free forum by Nabble | Edit this page |