Rename TJSONProtocol.java -> TSimpleJSONProtocol.java
Summary:
Chad Walters is developing a full-featured JSON-based protocol for Java,
so we're renaming the existing protocol to something more descriptive.
I also edited the file to rename the class and add to the header comment.
I have no idea if Subversion does explicit rename tracking, or if
git svn is smart enough to mark renames as such, but I would predict
"yes" to both. Let's find out.
Reviewed By: mcslee, jssarma
Test Plan: Ran "ant" in lib/java
Revert Plan: ok
git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665492 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/lib/java/src/protocol/TJSONProtocol.java b/lib/java/src/protocol/TSimpleJSONProtocol.java
similarity index 96%
rename from lib/java/src/protocol/TJSONProtocol.java
rename to lib/java/src/protocol/TSimpleJSONProtocol.java
index fefa65c..8cb3b69 100644
--- a/lib/java/src/protocol/TJSONProtocol.java
+++ b/lib/java/src/protocol/TSimpleJSONProtocol.java
@@ -16,17 +16,21 @@
/**
* JSON protocol implementation for thrift.
*
+ * This protocol is write-only and produces a simple output format
+ * suitable for parsing by scripting languages. It should not be
+ * confused with the full-featured TJSONProtocol.
+ *
* @author Joydeep Sen Sarma <jssarma@facebook.com>
* @author Mark Slee <mcslee@facebook.com>
*/
-public class TJSONProtocol extends TProtocol {
+public class TSimpleJSONProtocol extends TProtocol {
/**
* Factory
*/
public static class Factory implements TProtocolFactory {
public TProtocol getProtocol(TTransport trans) {
- return new TJSONProtocol(trans);
+ return new TSimpleJSONProtocol(trans);
}
}
@@ -99,7 +103,7 @@
/**
* Constructor
*/
- public TJSONProtocol(TTransport trans) {
+ public TSimpleJSONProtocol(TTransport trans) {
super(trans);
}