shunping commented on PR #25743: URL: https://github.com/apache/beam/pull/25743#issuecomment-3687289158
Found another one:) ``` WARNING:apache_beam.transforms.core:Using yield and return in the process method of <class 'apache_beam.io.fileio._ReadMatchesFn'> can lead to unexpected behavior, see:https://github.com/apache/beam/issues/22969. ``` https://github.com/apache/beam/blob/ca27526d759ea617166958940671019dea6547e2/sdks/python/apache_beam/io/fileio.py#L233 This looks like harmless as the previous one. Maybe it is better to only give a warning if yield and return in the same function AND some of the returns are returning something? -- 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]
