Re: RFR: 8255964: Add jcmd Thread.print to jtreg timeout handler [v2]

2020-11-06 Thread Nils Eliasson
> This patch adds jcmd Thread.print to the jtreg timeout handler. > > Please review. Nils Eliasson has updated the pull request incrementally with one additional commit since the last revision: add extended printing to jstack - Changes: - all: https://git.openjdk.java.net/jdk/

Re: RFR: 8255964: Add jcmd Thread.print to jtreg timeout handler

2020-11-06 Thread Nils Eliasson
On Fri, 6 Nov 2020 15:53:35 GMT, Igor Ignatyev wrote: >> This patch adds jcmd Thread.print to the jtreg timeout handler. >> >> Please review. > > Hi Nils, > > It looks alright, but could you please elaborate on why we need it when there > is already `jstack` action? > > — Igor They have the

Re: RFR: 8255964: Add jcmd Thread.print to jtreg timeout handler

2020-11-06 Thread Igor Ignatyev
On Thu, 5 Nov 2020 17:09:58 GMT, Nils Eliasson wrote: > This patch adds jcmd Thread.print to the jtreg timeout handler. > > Please review. Hi Nils, It looks alright, but could you please elaborate on why we need it when there is already `jstack` action? — Igor - PR: https://git

RFR: 8255964: Add jcmd Thread.print to jtreg timeout handler

2020-11-05 Thread Nils Eliasson
This patch adds jcmd Thread.print to the jtreg timeout handler. Please review. - Commit messages: - Add jcmd thread.print to timeout handler Changes: https://git.openjdk.java.net/jdk/pull/1080/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1080&range=00 Issue: https