You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "Polber (via GitHub)" <gi...@apache.org> on 2024/08/27 19:01:03 UTC

[I] [Bug]: Add check for self-referencing input in YAML transform [beam]

Polber opened a new issue, #32339:
URL: https://github.com/apache/beam/issues/32339

   ### What happened?
   
   The following pipeline will fail
   ```
   pipeline:
     transforms:
       - type: Create
         name: Source
         config:
           elements:
             - id: 1
         input: Source
       - type: LogForTesting
         input: Source
   ```
   with following error:
   ```
     ...
     File "/Users/jkinard/beam/sdks/python/apache_beam/yaml/yaml_transform.py", line 163, in strip_metadata
       if isinstance(spec, Mapping):
          ^^^^^^^^^^^^^^^^^^^^^^^^^
     File "/Users/jkinard/.pyenv/versions/3.11.6/lib/python3.11/typing.py", line 1305, in __instancecheck__
       return self.__subclasscheck__(type(obj))
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     File "/Users/jkinard/.pyenv/versions/3.11.6/lib/python3.11/typing.py", line 1583, in __subclasscheck__
       return issubclass(cls, self.__origin__)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   RecursionError: maximum recursion depth exceeded in __subclasscheck__
   ```
   
   Due to self-referencing transform - this should be a more clear error and caught earlier
   
   ### Issue Priority
   
   Priority: 2 (default / most bugs should be filed as P2)
   
   ### Issue Components
   
   - [ ] Component: Python SDK
   - [ ] Component: Java SDK
   - [ ] Component: Go SDK
   - [ ] Component: Typescript SDK
   - [ ] Component: IO connector
   - [X] Component: Beam YAML
   - [ ] Component: Beam examples
   - [ ] Component: Beam playground
   - [ ] Component: Beam katas
   - [ ] Component: Website
   - [ ] Component: Infrastructure
   - [ ] Component: Spark Runner
   - [ ] Component: Flink Runner
   - [ ] Component: Samza Runner
   - [ ] Component: Twister2 Runner
   - [ ] Component: Hazelcast Jet Runner
   - [ ] Component: Google Cloud Dataflow Runner


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug]: Add check for self-referencing input in YAML transform [beam]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed issue #32339: [Bug]: Add check for self-referencing input in YAML transform
URL: https://github.com/apache/beam/issues/32339


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug]: Add check for self-referencing input in YAML transform [beam]

Posted by "mravi (via GitHub)" <gi...@apache.org>.
mravi commented on issue #32339:
URL: https://github.com/apache/beam/issues/32339#issuecomment-2558236424

   .close-issue


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug]: Add check for self-referencing input in YAML transform [beam]

Posted by "mravi (via GitHub)" <gi...@apache.org>.
mravi commented on issue #32339:
URL: https://github.com/apache/beam/issues/32339#issuecomment-2496409675

   .take-issue


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug]: Add check for self-referencing input in YAML transform [beam]

Posted by "mravi (via GitHub)" <gi...@apache.org>.
mravi commented on issue #32339:
URL: https://github.com/apache/beam/issues/32339#issuecomment-2496394280

   .take_issue


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug]: Add check for self-referencing input in YAML transform [beam]

Posted by "mravi (via GitHub)" <gi...@apache.org>.
mravi commented on issue #32339:
URL: https://github.com/apache/beam/issues/32339#issuecomment-2498725055

   @Polber ptal https://github.com/apache/beam/pull/33208 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org