nikitagrover19 commented on issue #35095: URL: https://github.com/apache/beam/issues/35095#issuecomment-3615912172
Hi @hjtran ! I'd like to work on this issue as my first contribution to Apache Beam. I read through the ticket and took a quick look at the current implementation. I agree with the reasoning here — 'pvalue.py' mostly contains abstractions around pipeline values, while `Row` is a concrete, user-facing structure, so its placement does feel a bit out of place. Moving it into 'core.py' with a compatibility alias in 'pvalue.py' seems like the right general direction. At a first glance, this looks doable, but I want to spend some time going through the code layout and imports properly before making any changes, just to make sure nothing unexpected breaks. Let me know if there’s anything specific I should keep in mind as I dive deeper into it. Thanks! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
