Seems fair to me. Sam or Alex Goncharuk, can any of you review this as well?
--Yakov 2015-12-17 13:39 GMT+03:00 Anton Vinogradov <[email protected]>: > Yakov, > TC seems to be ok, could you please review > https://github.com/apache/ignite/pull/345/files before comit? > > On Thu, Dec 17, 2015 at 1:27 PM, Yakov Zhdanov <[email protected]> > wrote: > >> Anton, can you please let us know if changes have been merged? >> >> --Yakov >> >> 2015-12-16 17:00 GMT+03:00 Anton Vinogradov <[email protected]>: >> >>> Val, >>> Yes, Please check my pull-request >>> https://github.com/apache/ignite/pull/345 >>> I'll megre changes tomorrow morning in case everything is ok & TC passed. >>> >>> On Wed, Dec 16, 2015 at 6:11 AM, vkulichenko < >>> [email protected]> wrote: >>> >>>> Completely agree with Juan. Setting class loader only for default >>>> configuration is definitely not enough for the most use cases. >>>> >>>> I reopened the ticket. Anton, will you have a chance to finish the fix? >>>> >>>> -Val >>>> >>>> >>>> >>>> -- >>>> View this message in context: >>>> http://apache-ignite-users.70518.x6.nabble.com/Help-with-integrating-Ignite-as-JCache-with-JBoss-EAP-6-4-tp2134p2228.html >>>> Sent from the Apache Ignite Users mailing list archive at Nabble.com. >>>> >>> >>> >> >
