You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@avro.apache.org by fo...@apache.org on 2019/10/11 08:38:03 UTC
[avro] branch master updated: AVRO-2581:Generate @Deprecated public
attribute in Java (#669)
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 <51...@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;