Re: Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56934/#review166431 --- Ship it! Ship It! - Anthony Baker On Feb. 22, 2017, 9:39 p.m

Re: Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56934/ --- (Updated Feb. 22, 2017, 9:39 p.m.) Review request for geode, Anthony Baker, Jin

Re: Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Kirk Lund
> On Feb. 22, 2017, 8:11 p.m., Anthony Baker wrote: > > I'll close my PR in favor of these changes. However, there are some NOTICE > > file updates needed (e.g. see > > https://github.com/apache/geode/pull/403/files). I already approved your PR. We could merge your PR in first? - Kirk ---

Re: Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Kirk Lund
> On Feb. 22, 2017, 6:02 p.m., Mark Bretl wrote: > > Have new versions been checked for any license changes? > > Also, I think Anthony has an outstanding pull request for updating the > > Spring Framework version. I added the license for each version in the description. Looks like they all sta

Re: Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56934/ --- (Updated Feb. 22, 2017, 9:05 p.m.) Review request for geode, Anthony Baker, Jin

Re: Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56934/#review166406 --- I'll close my PR in favor of these changes. However, there are so

Re: Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Mark Bretl
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56934/#review166382 --- Have new versions been checked for any license changes? Also, I th

Review Request 56934: GEODE-2460: update dependency versions

2017-02-22 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56934/ --- Review request for geode, Anthony Baker, Jinmei Liao, Jared Stewart, Kevin Dulin