You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by "tvalentyn (via GitHub)" <gi...@apache.org> on 2024/07/12 17:14:12 UTC

[I] [Feature Request]: Tour of Beam should use the beam.LogElements transform instead of defining `class Output(beam.PTransform):` [beam]

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

   ### What would you like to happen?
   
   All tour of Beam examples I came across include
   
   ```
   # Output PCollection
   class Output(beam.PTransform):
       class _OutputFn(beam.DoFn):
           def __init__(self, prefix=''):
               super().__init__()
               self.prefix = prefix
   
           def process(self, element):
               print(self.prefix+str(element))
   
       def __init__(self, label=None,prefix=''):
           super().__init__(label)
           self.prefix = prefix
   
       def expand(self, input):
           input | beam.ParDo(self._OutputFn(self.prefix))
   
   ```
   
   This adds unnecessary complexity and verbosity and reimplements beam.LogElements: https://github.com/apache/beam/blob/b573c8ffc8b7ac765ff3ef281a5c6a5dd3d084ef/sdks/python/apache_beam/transforms/util.py#L1303
   
   ### Issue Priority
   
   Priority: 3 (nice-to-have improvement)
   
   ### Issue Components
   
   - [ ] Component: Python SDK
   - [ ] Component: Java SDK
   - [ ] Component: Go SDK
   - [ ] Component: Typescript SDK
   - [ ] Component: IO connector
   - [ ] Component: Beam YAML
   - [X] Component: Beam examples
   - [ ] Component: Beam playground
   - [ ] Component: Beam katas
   - [ ] Component: Website
   - [ ] 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] [Feature Request]: Tour of Beam should use the beam.LogElements transform instead of defining `class Output(beam.PTransform):` [beam]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.

github-actions[bot] commented on issue #31867:
URL: https://github.com/apache/beam/issues/31867#issuecomment-3401640027

   This issue has been marked as stale due to 150 days of inactivity. It will be closed in 30 days if no further activity occurs. If you think that’s incorrect or this issue still needs to be addressed, please simply write any comment. If closed, you can reopen the issue at any time. Thank you for your contributions.


-- 
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] [Feature Request]: Tour of Beam should use the beam.LogElements transform instead of defining `class Output(beam.PTransform):` [beam]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.

github-actions[bot] commented on issue #31867:
URL: https://github.com/apache/beam/issues/31867#issuecomment-3532578430

   This issue has been closed due to lack of activity. If you think that is incorrect, you can reopen the issue at any time.


-- 
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] [Feature Request]: Tour of Beam should use the beam.LogElements transform instead of defining `class Output(beam.PTransform):` [beam]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.

github-actions[bot] closed issue #31867: [Feature Request]: Tour of Beam should use the beam.LogElements transform instead of defining `class Output(beam.PTransform):`
URL: https://github.com/apache/beam/issues/31867


-- 
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