This is an automated email from the ASF dual-hosted git repository. dlmarion pushed a commit to branch 2.1 in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/2.1 by this push: new e88583f681 Regenerated protobuf files with protoc 25.6 (#5467) e88583f681 is described below commit e88583f681f6ef1d9f1999c44d06feba509c1d4c Author: Dave Marion <dlmar...@apache.org> AuthorDate: Thu Apr 10 15:09:22 2025 -0400 Regenerated protobuf files with protoc 25.6 (#5467) https://github.com/protocolbuffers/protobuf/pull/20084 introduced an exception when a recent protobuf-java library processed a protobuf generated object that was created with an older version of protoc. Regenerated the protobuf files so that users that are using replication don't run into this error. Closes #5462 --- .../server/replication/proto/Replication.java | 197 ++++++++++----------- server/base/src/main/scripts/generate-protobuf.sh | 2 +- 2 files changed, 91 insertions(+), 108 deletions(-) diff --git a/server/base/src/main/java/org/apache/accumulo/server/replication/proto/Replication.java b/server/base/src/main/java/org/apache/accumulo/server/replication/proto/Replication.java index 3a5c9ebbed..65d38895b6 100644 --- a/server/base/src/main/java/org/apache/accumulo/server/replication/proto/Replication.java +++ b/server/base/src/main/java/org/apache/accumulo/server/replication/proto/Replication.java @@ -19,6 +19,7 @@ // Generated by the protocol buffer compiler. DO NOT EDIT! // source: src/main/protobuf/replication.proto +// Protobuf Java Version: 3.25.6 package org.apache.accumulo.server.replication.proto; @SuppressWarnings("unused") public final class Replication { @@ -153,74 +154,6 @@ package org.apache.accumulo.server.replication.proto; return new Status(); } - @java.lang.Override - public final com.google.protobuf.UnknownFieldSet - getUnknownFields() { - return this.unknownFields; - } - private Status( - com.google.protobuf.CodedInputStream input, - com.google.protobuf.ExtensionRegistryLite extensionRegistry) - throws com.google.protobuf.InvalidProtocolBufferException { - this(); - if (extensionRegistry == null) { - throw new java.lang.NullPointerException(); - } - int mutable_bitField0_ = 0; - com.google.protobuf.UnknownFieldSet.Builder unknownFields = - com.google.protobuf.UnknownFieldSet.newBuilder(); - try { - boolean done = false; - while (!done) { - int tag = input.readTag(); - switch (tag) { - case 0: - done = true; - break; - case 8: { - bitField0_ |= 0x00000001; - begin_ = input.readInt64(); - break; - } - case 16: { - bitField0_ |= 0x00000002; - end_ = input.readInt64(); - break; - } - case 24: { - bitField0_ |= 0x00000004; - infiniteEnd_ = input.readBool(); - break; - } - case 32: { - bitField0_ |= 0x00000008; - closed_ = input.readBool(); - break; - } - case 40: { - bitField0_ |= 0x00000010; - createdTime_ = input.readInt64(); - break; - } - default: { - if (!parseUnknownField( - input, unknownFields, extensionRegistry, tag)) { - done = true; - } - break; - } - } - } - } catch (com.google.protobuf.InvalidProtocolBufferException e) { - throw e.setUnfinishedMessage(this); - } catch (java.io.IOException e) { - throw new com.google.protobuf.InvalidProtocolBufferException( - e).setUnfinishedMessage(this); - } finally { - this.unknownFields = unknownFields.build(); - makeExtensionsImmutable(); - } - } public static final com.google.protobuf.Descriptors.Descriptor getDescriptor() { return org.apache.accumulo.server.replication.proto.Replication.internal_static_Status_descriptor; @@ -236,7 +169,7 @@ package org.apache.accumulo.server.replication.proto; private int bitField0_; public static final int BEGIN_FIELD_NUMBER = 1; - private long begin_; + private long begin_ = 0L; /** * <pre> * offset where replication should start @@ -263,7 +196,7 @@ package org.apache.accumulo.server.replication.proto; } public static final int END_FIELD_NUMBER = 2; - private long end_; + private long end_ = 0L; /** * <pre> * offset where data is ready for replication @@ -290,7 +223,7 @@ package org.apache.accumulo.server.replication.proto; } public static final int INFINITEEND_FIELD_NUMBER = 3; - private boolean infiniteEnd_; + private boolean infiniteEnd_ = false; /** * <pre> * do we have a discrete 'end' @@ -317,7 +250,7 @@ package org.apache.accumulo.server.replication.proto; } public static final int CLOSED_FIELD_NUMBER = 4; - private boolean closed_; + private boolean closed_ = false; /** * <pre> * will more data be appended to the file @@ -344,7 +277,7 @@ package org.apache.accumulo.server.replication.proto; } public static final int CREATEDTIME_FIELD_NUMBER = 5; - private long createdTime_; + private long createdTime_ = 0L; /** * <pre> * when, in ms, was the file created? @@ -399,7 +332,7 @@ package org.apache.accumulo.server.replication.proto; if (((bitField0_ & 0x00000010) != 0)) { output.writeInt64(5, createdTime_); } - unknownFields.writeTo(output); + getUnknownFields().writeTo(output); } @java.lang.Override @@ -428,7 +361,7 @@ package org.apache.accumulo.server.replication.proto; size += com.google.protobuf.CodedOutputStream .computeInt64Size(5, createdTime_); } - size += unknownFields.getSerializedSize(); + size += getUnknownFields().getSerializedSize(); memoizedSize = size; return size; } @@ -468,11 +401,10 @@ package org.apache.accumulo.server.replication.proto; if (getCreatedTime() != other.getCreatedTime()) return false; } - if (!unknownFields.equals(other.unknownFields)) return false; + if (!getUnknownFields().equals(other.getUnknownFields())) return false; return true; } - @SuppressWarnings("unchecked") @java.lang.Override public int hashCode() { if (memoizedHashCode != 0) { @@ -505,7 +437,7 @@ package org.apache.accumulo.server.replication.proto; hash = (53 * hash) + com.google.protobuf.Internal.hashLong( getCreatedTime()); } - hash = (29 * hash) + unknownFields.hashCode(); + hash = (29 * hash) + getUnknownFields().hashCode(); memoizedHashCode = hash; return hash; } @@ -554,11 +486,13 @@ package org.apache.accumulo.server.replication.proto; return com.google.protobuf.GeneratedMessageV3 .parseWithIOException(PARSER, input, extensionRegistry); } + public static org.apache.accumulo.server.replication.proto.Replication.Status parseDelimitedFrom(java.io.InputStream input) throws java.io.IOException { return com.google.protobuf.GeneratedMessageV3 .parseDelimitedWithIOException(PARSER, input); } + public static org.apache.accumulo.server.replication.proto.Replication.Status parseDelimitedFrom( java.io.InputStream input, com.google.protobuf.ExtensionRegistryLite extensionRegistry) @@ -622,32 +556,23 @@ package org.apache.accumulo.server.replication.proto; // Construct using org.apache.accumulo.server.replication.proto.Replication.Status.newBuilder() private Builder() { - maybeForceBuilderInitialization(); + } private Builder( com.google.protobuf.GeneratedMessageV3.BuilderParent parent) { super(parent); - maybeForceBuilderInitialization(); - } - private void maybeForceBuilderInitialization() { - if (com.google.protobuf.GeneratedMessageV3 - .alwaysUseFieldBuilders) { - } + } @java.lang.Override public Builder clear() { super.clear(); + bitField0_ = 0; begin_ = 0L; - bitField0_ = (bitField0_ & ~0x00000001); end_ = 0L; - bitField0_ = (bitField0_ & ~0x00000002); infiniteEnd_ = false; - bitField0_ = (bitField0_ & ~0x00000004); closed_ = false; - bitField0_ = (bitField0_ & ~0x00000008); createdTime_ = 0L; - bitField0_ = (bitField0_ & ~0x00000010); return this; } @@ -674,6 +599,12 @@ package org.apache.accumulo.server.replication.proto; @java.lang.Override public org.apache.accumulo.server.replication.proto.Replication.Status buildPartial() { org.apache.accumulo.server.replication.proto.Replication.Status result = new org.apache.accumulo.server.replication.proto.Replication.Status(this); + if (bitField0_ != 0) { buildPartial0(result); } + onBuilt(); + return result; + } + + private void buildPartial0(org.apache.accumulo.server.replication.proto.Replication.Status result) { int from_bitField0_ = bitField0_; int to_bitField0_ = 0; if (((from_bitField0_ & 0x00000001) != 0)) { @@ -696,9 +627,7 @@ package org.apache.accumulo.server.replication.proto; result.createdTime_ = createdTime_; to_bitField0_ |= 0x00000010; } - result.bitField0_ = to_bitField0_; - onBuilt(); - return result; + result.bitField0_ |= to_bitField0_; } @java.lang.Override @@ -760,7 +689,7 @@ package org.apache.accumulo.server.replication.proto; if (other.hasCreatedTime()) { setCreatedTime(other.getCreatedTime()); } - this.mergeUnknownFields(other.unknownFields); + this.mergeUnknownFields(other.getUnknownFields()); onChanged(); return this; } @@ -775,17 +704,55 @@ package org.apache.accumulo.server.replication.proto; com.google.protobuf.CodedInputStream input, com.google.protobuf.ExtensionRegistryLite extensionRegistry) throws java.io.IOException { - org.apache.accumulo.server.replication.proto.Replication.Status parsedMessage = null; + if (extensionRegistry == null) { + throw new java.lang.NullPointerException(); + } try { - parsedMessage = PARSER.parsePartialFrom(input, extensionRegistry); + boolean done = false; + while (!done) { + int tag = input.readTag(); + switch (tag) { + case 0: + done = true; + break; + case 8: { + begin_ = input.readInt64(); + bitField0_ |= 0x00000001; + break; + } // case 8 + case 16: { + end_ = input.readInt64(); + bitField0_ |= 0x00000002; + break; + } // case 16 + case 24: { + infiniteEnd_ = input.readBool(); + bitField0_ |= 0x00000004; + break; + } // case 24 + case 32: { + closed_ = input.readBool(); + bitField0_ |= 0x00000008; + break; + } // case 32 + case 40: { + createdTime_ = input.readInt64(); + bitField0_ |= 0x00000010; + break; + } // case 40 + default: { + if (!super.parseUnknownField(input, extensionRegistry, tag)) { + done = true; // was an endgroup tag + } + break; + } // default: + } // switch (tag) + } // while (!done) } catch (com.google.protobuf.InvalidProtocolBufferException e) { - parsedMessage = (org.apache.accumulo.server.replication.proto.Replication.Status) e.getUnfinishedMessage(); throw e.unwrapIOException(); } finally { - if (parsedMessage != null) { - mergeFrom(parsedMessage); - } - } + onChanged(); + } // finally return this; } private int bitField0_; @@ -825,8 +792,9 @@ package org.apache.accumulo.server.replication.proto; * @return This builder for chaining. */ public Builder setBegin(long value) { - bitField0_ |= 0x00000001; + begin_ = value; + bitField0_ |= 0x00000001; onChanged(); return this; } @@ -880,8 +848,9 @@ package org.apache.accumulo.server.replication.proto; * @return This builder for chaining. */ public Builder setEnd(long value) { - bitField0_ |= 0x00000002; + end_ = value; + bitField0_ |= 0x00000002; onChanged(); return this; } @@ -935,8 +904,9 @@ package org.apache.accumulo.server.replication.proto; * @return This builder for chaining. */ public Builder setInfiniteEnd(boolean value) { - bitField0_ |= 0x00000004; + infiniteEnd_ = value; + bitField0_ |= 0x00000004; onChanged(); return this; } @@ -990,8 +960,9 @@ package org.apache.accumulo.server.replication.proto; * @return This builder for chaining. */ public Builder setClosed(boolean value) { - bitField0_ |= 0x00000008; + closed_ = value; + bitField0_ |= 0x00000008; onChanged(); return this; } @@ -1045,8 +1016,9 @@ package org.apache.accumulo.server.replication.proto; * @return This builder for chaining. */ public Builder setCreatedTime(long value) { - bitField0_ |= 0x00000010; + createdTime_ = value; + bitField0_ |= 0x00000010; onChanged(); return this; } @@ -1097,7 +1069,18 @@ package org.apache.accumulo.server.replication.proto; com.google.protobuf.CodedInputStream input, com.google.protobuf.ExtensionRegistryLite extensionRegistry) throws com.google.protobuf.InvalidProtocolBufferException { - return new Status(input, extensionRegistry); + Builder builder = newBuilder(); + try { + builder.mergeFrom(input, extensionRegistry); + } catch (com.google.protobuf.InvalidProtocolBufferException e) { + throw e.setUnfinishedMessage(builder.buildPartial()); + } catch (com.google.protobuf.UninitializedMessageException e) { + throw e.asInvalidProtocolBufferException().setUnfinishedMessage(builder.buildPartial()); + } catch (java.io.IOException e) { + throw new com.google.protobuf.InvalidProtocolBufferException(e) + .setUnfinishedMessage(builder.buildPartial()); + } + return builder.buildPartial(); } }; diff --git a/server/base/src/main/scripts/generate-protobuf.sh b/server/base/src/main/scripts/generate-protobuf.sh index 36884e0f39..7c83d68b95 100755 --- a/server/base/src/main/scripts/generate-protobuf.sh +++ b/server/base/src/main/scripts/generate-protobuf.sh @@ -25,7 +25,7 @@ # In other scripts, set the variables that diverge from the defaults below, then call this script. # Leave the BUILD_DIR and FINAL_DIR alone for Maven builds. # ======================================================================================================================== -[[ -z $REQUIRED_PROTOC_VERSION ]] && REQUIRED_PROTOC_VERSION='libprotoc 3.19.2' +[[ -z $REQUIRED_PROTOC_VERSION ]] && REQUIRED_PROTOC_VERSION='libprotoc 25.6' [[ -z $BUILD_DIR ]] && BUILD_DIR='target/proto' [[ -z $FINAL_DIR ]] && FINAL_DIR='src/main' # ========================================================================================================================