[ 
https://issues.apache.org/jira/browse/BEAM-10265?focusedWorklogId=764276&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-764276
 ]

ASF GitHub Bot logged work on BEAM-10265:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 29/Apr/22 14:41
            Start Date: 29/Apr/22 14:41
    Worklog Time Spent: 10m 
      Work Description: andreigurau commented on code in PR #17477:
URL: https://github.com/apache/beam/pull/17477#discussion_r861871811


##########
sdks/java/extensions/protobuf/src/main/java/org/apache/beam/sdk/extensions/protobuf/ProtoSchemaTranslator.java:
##########
@@ -156,6 +165,16 @@ static Schema getSchema(Class<? extends Message> clazz) {
   }
 
   static Schema getSchema(Descriptors.Descriptor descriptor) {

Review Comment:
   Resolving by marking the method as synchronized, and removing the javadoc





Issue Time Tracking
-------------------

    Worklog Id:     (was: 764276)
    Time Spent: 3h  (was: 2h 50m)

> GetterBasedSchemaProvider#schemaFor stack overflows when given a recursive 
> schema
> ---------------------------------------------------------------------------------
>
>                 Key: BEAM-10265
>                 URL: https://issues.apache.org/jira/browse/BEAM-10265
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-java-core
>    Affects Versions: 2.21.0, 2.22.0
>            Reporter: Reza ardeshir rokni
>            Assignee: Andrei Gurau
>            Priority: P3
>              Labels: Clarified, starter
>          Time Spent: 3h
>  Remaining Estimate: 0h
>
> Proto:
> message TSFoo {
>  string a = 1;
>  string b = 2;
>  TSFoo theOlderMe = 3;
> }
> new ProtoMessageSchema().schemaFor(TypeDescriptor.of(Foo.TSFoo.class));
> Causes a stackoverflow. 
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to