----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55280/#review160793 -----------------------------------------------------------
Ship it! Will there be another changeset to remove the code from geode-json sub-project as well? - Mark Bretl On Jan. 6, 2017, 1:09 p.m., Anthony Baker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55280/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2017, 1:09 p.m.) > > > Review request for geode, Jinmei Liao, Kirk Lund, and Mark Bretl. > > > Repository: geode > > > Description > ------- > > GEODE-2142: Add JSON usage to NOTICE > > GEODE-2142: Remove JSON dependency from pulse > > Remove the Cat-X licensed JSON software from geode-pulse and replace > it with the friendly open-json replacement. See > https://github.com/tdunning/open-json. > > Update copyright date in NOTICE > > > Diffs > ----- > > NOTICE 4801b2aedf4f254c85cba9f27d2f99d2ca2e046b > geode-assembly/src/main/dist/NOTICE > 83d0f3313a01881c9c651077036c17f962cd38b9 > geode-pulse/build.gradle 94f26f7a4106ea085ba1bc9a7001967f6b47a863 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/CDL.java > 6243f8f599342dbdf00d3b541dfeca9da5def508 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/Cookie.java > 1a0fa6fb7c1edd8b81bd4513ea6786a8ba4ba59c > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/CookieList.java > 99498f500fec9d85a30b610564c706731a55ed68 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/HTTP.java > 234d2bddc483f71503f84886be949f2b93a7ddd6 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/HTTPTokener.java > 09b68d872f492f65cff14a02997dbae657027992 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONArray.java > 0db27363e51eaa4e7d40724ea82ed29975df46c4 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONException.java > fbd8866cd59817b588b84ae3bceb6e39f49fa8e2 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONML.java > fc19ca549da14441c8895bce56d46bf880b13c90 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONObject.java > 13f2ccad8c24a346ef8eb20cd195f5bf53211eeb > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONString.java > 45585c75dbe5c0dd50359117713e078263dfb7b3 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONStringer.java > 8c5ff0826d5f6ad998d39ee73ef4a255e5fc0423 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONTokener.java > 0f4b7ee8f017c1f561f954dab1e409236d382bf2 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/JSONWriter.java > dea900ab240b9a1b25079dec43c1e775e7caef76 > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/README > 2d7cdcf8a0ccdd29e0b1a3337da4338a51649bce > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/XML.java > 25675f89b6d2e22fced09dd366b18753f28b3ab7 > > geode-pulse/src/main/java/org/apache/geode/tools/pulse/internal/json/XMLTokener.java > ff81731e11b49b649b92580fd169440c734a1006 > geode-pulse/src/main/webapp/META-INF/NOTICE > 706eda265cf0b54c73d01df85c0a79bde9082d8a > > geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/junit/BaseServiceTest.java > ccc147d1533142d54c8c7378c935da8f5e3234f5 > > geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/junit/ClusterSelectedRegionServiceTest.java > 77e6dd89340c59f7c15363a1d878ca3b6165d90a > > geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/junit/ClusterSelectedRegionsMemberServiceTest.java > 512cfc78e811a3234f170b243cf3983a96fc4788 > > geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/junit/MemberGatewayHubServiceTest.java > 980a772a6a43a60befe3f4cec2c898b3e62ad038 > geode-web-api/src/main/webapp/META-INF/NOTICE > 21082b6de4fbd7ba4ec4bcb61e9e1bfda5ac162a > geode-web/src/main/webapp/META-INF/NOTICE > 8c6e1d7faf3188599305f44c44318f6e6cfd00e6 > gradle/dependency-versions.properties > 828b0882d0813919e699d1ec83c7ba9c59f0220e > > Diff: https://reviews.apache.org/r/55280/diff/ > > > Testing > ------- > > > Thanks, > > Anthony Baker > >
