[avro] branch master updated: AVRO-2581:Generate @Deprecated public attribute in Java (#669)

[fokko]

This is an automated email from the ASF dual-hosted git repository.

fokko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/avro.git


The following commit(s) were added to refs/heads/master by this push:
     new 2cd197a  AVRO-2581:Generate @Deprecated public attribute in Java (#669)
2cd197a is described below

commit 2cd197acd8295f949f429613b82b66d042c43342
Author: Zezeng Wang <51382...@qq.com>
AuthorDate: Fri Oct 11 16:37:58 2019 +0800

    AVRO-2581:Generate @Deprecated public attribute in Java (#669)
---
 .../src/test/java/org/apache/avro/TestProtocolSpecific.java  |  2 +-
 .../apache/avro/compiler/specific/TestSpecificCompiler.java  |  2 +-
 .../org/apache/avro/specific/TestSpecificRecordBuilder.java  | 12 ++++++------
 .../src/main/java/org/apache/avro/mojo/AbstractAvroMojo.java |  2 +-
 4 files changed, 9 insertions(+), 9 deletions(-)

diff --git 
a/lang/java/ipc/src/test/java/org/apache/avro/TestProtocolSpecific.java 
b/lang/java/ipc/src/test/java/org/apache/avro/TestProtocolSpecific.java
index c66cedf..f53106f 100644
--- a/lang/java/ipc/src/test/java/org/apache/avro/TestProtocolSpecific.java
+++ b/lang/java/ipc/src/test/java/org/apache/avro/TestProtocolSpecific.java
@@ -298,7 +298,7 @@ public class TestProtocolSpecific {
       }
 
       // check that a given client protocol is only sent once
-      String clientProtocol = context.getHandshakeRequest().clientProtocol;
+      String clientProtocol = 
context.getHandshakeRequest().getClientProtocol();
       if (clientProtocol != null) {
         assertFalse(seenProtocols.contains(clientProtocol));
         seenProtocols.add(clientProtocol);
diff --git 
a/lang/java/ipc/src/test/java/org/apache/avro/compiler/specific/TestSpecificCompiler.java
 
b/lang/java/ipc/src/test/java/org/apache/avro/compiler/specific/TestSpecificCompiler.java
index 559bff5..32b2edc 100644
--- 
a/lang/java/ipc/src/test/java/org/apache/avro/compiler/specific/TestSpecificCompiler.java
+++ 
b/lang/java/ipc/src/test/java/org/apache/avro/compiler/specific/TestSpecificCompiler.java
@@ -581,7 +581,7 @@ public class TestSpecificCompiler {
     assertNotNull(Kind.class.getAnnotation(TestAnnotation.class));
 
     // a field
-    
assertNotNull(TestRecord.class.getField("name").getAnnotation(TestAnnotation.class));
+    
assertNotNull(TestRecord.class.getDeclaredField("name").getAnnotation(TestAnnotation.class));
     // a method
     
assertNotNull(Simple.class.getMethod("ack").getAnnotation(TestAnnotation.class));
   }
diff --git 
a/lang/java/ipc/src/test/java/org/apache/avro/specific/TestSpecificRecordBuilder.java
 
b/lang/java/ipc/src/test/java/org/apache/avro/specific/TestSpecificRecordBuilder.java
index 708c68c..867876a 100644
--- 
a/lang/java/ipc/src/test/java/org/apache/avro/specific/TestSpecificRecordBuilder.java
+++ 
b/lang/java/ipc/src/test/java/org/apache/avro/specific/TestSpecificRecordBuilder.java
@@ -214,12 +214,12 @@ public class TestSpecificRecordBuilder {
     long startTimeNanos = System.nanoTime();
     for (int ii = 0; ii < count; ii++) {
       Person person = new Person();
-      person.name = "James Gosling";
-      person.year_of_birth = 1955;
-      person.state = "CA";
-      person.country = "US";
-      person.friends = friends;
-      person.languages = languages;
+      person.setName("James Gosling");
+      person.setYearOfBirth(1955);
+      person.setState("CA");
+      person.setCountry("US");
+      person.setFriends(friends);
+      person.setLanguages(languages);
     }
     long durationNanos = System.nanoTime() - startTimeNanos;
     double durationMillis = durationNanos / 1e6d;
diff --git 
a/lang/java/maven-plugin/src/main/java/org/apache/avro/mojo/AbstractAvroMojo.java
 
b/lang/java/maven-plugin/src/main/java/org/apache/avro/mojo/AbstractAvroMojo.java
index 4d749ab..7cf96f1 100644
--- 
a/lang/java/maven-plugin/src/main/java/org/apache/avro/mojo/AbstractAvroMojo.java
+++ 
b/lang/java/maven-plugin/src/main/java/org/apache/avro/mojo/AbstractAvroMojo.java
@@ -72,7 +72,7 @@ public abstract class AbstractAvroMojo extends AbstractMojo {
    * string values of SpecificCompiler.FieldVisibility. The text is case
    * insensitive.
    *
-   * @parameter default-value="PUBLIC_DEPRECATED"
+   * @parameter default-value="PRIVATE"
    */
   private String fieldVisibility;