Skip to content

AVRO-4133: Support default enum value in Protobuf to Avro #3367

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
opwvhk merged 3 commits into from
Jun 10, 2025

[AVRO-4133] Exclude generated test proto code

8728c43
Select commit
Loading
Failed to load commit list.
Merged

AVRO-4133: Support default enum value in Protobuf to Avro #3367

[AVRO-4133] Exclude generated test proto code
8728c43
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL completed Jun 6, 2025 in 7s

3 configurations not found

Warning: Code scanning cannot determine the alerts introduced by this pull request, because 3 configurations present on refs/heads/main were not found:

Actions workflow (codeql-csharp-analysis.yml)

  • ❓  .github/workflows/codeql-csharp-analysis.yml:analyze/language:csharp

Actions workflow (codeql-js-analysis.yml)

  • ❓  .github/workflows/codeql-js-analysis.yml:analyze/language:javascript

Actions workflow (codeql-py-analysis.yml)

  • ❓  .github/workflows/codeql-py-analysis.yml:analyze/language:python

View all branch alerts.

Annotations

Check notice on line 18 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Useless parameter Note test

The parameter 'registry' is never used.

Check notice on line 60 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Missing Override annotation Note test

This method overrides
ProtocolMessageEnum.getNumber
Loading
; it is advisable to add an Override annotation.

Check notice on line 99 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Missing Override annotation Note test

This method overrides
EnumLiteMap.findValueByNumber
Loading
; it is advisable to add an Override annotation.

Check notice on line 104 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Missing Override annotation Note test

This method overrides
ProtocolMessageEnum.getValueDescriptor
Loading
; it is advisable to add an Override annotation.

Check notice on line 111 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Missing Override annotation Note test

This method overrides
ProtocolMessageEnum.getDescriptorForType
Loading
; it is advisable to add an Override annotation.

Check failure on line 469 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

No clone method Error test

No clone method, yet implements Cloneable.

Check notice on line 547 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Confusing overloading of methods Note test

Method Builder.mergeFrom(..) could be confused with overloaded method
Builder.mergeFrom
Loading
, since dispatch depends on static types.
Method Builder.mergeFrom(..) could be confused with overloaded method
mergeFrom
Loading
, since dispatch depends on static types.

Check notice on line 634 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Missing Override annotation Note test

This method overrides
UserOrBuilder.getName
Loading
; it is advisable to add an Override annotation.

Check notice on line 651 in lang/java/protobuf/src/test/java/org/apache/avro/protobuf/noopt/TestProto3.java

See this annotation in the file changed.

Code scanning / CodeQL

Missing Override annotation Note test

This method overrides
UserOrBuilder.getNameBytes
Loading
; it is advisable to add an Override annotation.