On 2015-11-25 17:53, Jaroslav Bachorik wrote:
On 25.11.2015 11:36, Magnus Ihse Bursie wrote:
On 2015-11-25 10:39, Alan Bateman wrote:
On 25/11/2015 09:25, Jaroslav Bachorik wrote:
I don't think we can just repackage these interfaces - they are
remote API and we would break compatibility (
> On Nov 26, 2015, at 2:12 AM, Jaroslav Bachorik
> wrote:
>
> Mandy,
>
> On 25.11.2015 18:22, Mandy Chung wrote:
>>
>>> On Nov 25, 2015, at 8:53 AM, Jaroslav Bachorik
>>> wrote:
>>>
>>> Updated webrevs:
>>>
>>> http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/top
>>> http://cr.open
On 25/11/2015 16:53, Jaroslav Bachorik wrote:
Updated webrevs:
http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/top
http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/jdk
Thanks to Erik J. for helping me with the build script!
Updated version looks good to me.
-Alan
Mandy,
On 25.11.2015 18:22, Mandy Chung wrote:
On Nov 25, 2015, at 8:53 AM, Jaroslav Bachorik
wrote:
Updated webrevs:
http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/top
http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/jdk
This looks fine to me. Is there any test referen
> On Nov 25, 2015, at 8:53 AM, Jaroslav Bachorik
> wrote:
>
> Updated webrevs:
>
> http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/top
> http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/jdk
>
This looks fine to me. Is there any test referencing the packages in
jdk.jvmstat.rm
Build changes look good to me now.
/Erik
On 2015-11-25 17:53, Jaroslav Bachorik wrote:
On 25.11.2015 11:36, Magnus Ihse Bursie wrote:
On 2015-11-25 10:39, Alan Bateman wrote:
On 25/11/2015 09:25, Jaroslav Bachorik wrote:
I don't think we can just repackage these interfaces - they are
remo
On 25/11/2015 09:25, Jaroslav Bachorik wrote:
I don't think we can just repackage these interfaces - they are remote
API and we would break compatibility (eg. JDK 8 jvmstat would not be
able to query JVM exposed via JDK 9 jstatd).
You're right. there might be an expectation on interop althou
On 25.11.2015 09:37, Erik Joelsson wrote:
On 2015-11-25 09:11, Staffan Larsen wrote:
On 24 nov. 2015, at 17:03, Jaroslav Bachorik
wrote:
Please, review the following change
Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
Webrevs:
* top level: http://cr.openjdk.java.net/~jbachorik/8
On 25.11.2015 11:36, Magnus Ihse Bursie wrote:
On 2015-11-25 10:39, Alan Bateman wrote:
On 25/11/2015 09:25, Jaroslav Bachorik wrote:
I don't think we can just repackage these interfaces - they are
remote API and we would break compatibility (eg. JDK 8 jvmstat would
not be able to query JVM
On 2015-11-25 09:11, Staffan Larsen wrote:
On 24 nov. 2015, at 17:03, Jaroslav Bachorik
wrote:
Please, review the following change
Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
Webrevs:
* top level: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/top
* jdk: http://cr.open
On 2015-11-25 10:39, Alan Bateman wrote:
On 25/11/2015 09:25, Jaroslav Bachorik wrote:
I don't think we can just repackage these interfaces - they are
remote API and we would break compatibility (eg. JDK 8 jvmstat would
not be able to query JVM exposed via JDK 9 jstatd).
You're right. there
On 2015-11-25 10:28, Jaroslav Bachorik wrote:
On 25.11.2015 09:37, Erik Joelsson wrote:
On 2015-11-25 09:11, Staffan Larsen wrote:
On 24 nov. 2015, at 17:03, Jaroslav Bachorik
wrote:
Please, review the following change
Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
Webrevs:
* t
On 25.11.2015 10:02, Alan Bateman wrote:
On 24/11/2015 16:03, Jaroslav Bachorik wrote:
Please, review the following change
Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
Webrevs:
* top level: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/top
* jdk: http://cr.openjdk.java.ne
On 24/11/2015 16:03, Jaroslav Bachorik wrote:
Please, review the following change
Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
Webrevs:
* top level: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/top
* jdk: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/jdk
This p
On 25/11/2015 08:11, Staffan Larsen wrote:
On 24 nov. 2015, at 17:03, Jaroslav Bachorik
wrote:
Please, review the following change
Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
Webrevs:
* top level: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/top
* jdk: http://cr.openjd
> On 24 nov. 2015, at 17:03, Jaroslav Bachorik
> wrote:
>
> Please, review the following change
>
> Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
> Webrevs:
> * top level: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/top
> * jdk: http://cr.openjdk.java.net/~jbachorik/8043
Please, review the following change
Issue : https://bugs.openjdk.java.net/browse/JDK-8043138
Webrevs:
* top level: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/top
* jdk: http://cr.openjdk.java.net/~jbachorik/8043138/webrev.00/jdk
This patch splits up the jdk.jvmstat module to jdk.jvm
17 matches
Mail list logo