robertwb commented on PR #17767:
URL: https://github.com/apache/beam/pull/17767#issuecomment-1138913150
Oh, didn't realize you were looking at this as well.
The primary advantage of this form is for anything that is difficult to
express as a lambda, e.g. having to write
```
PythonCallableSource.of(
"foo")
.withContext("def foo(...): ...")
```
feels a bit roundabout. I was also leaning towards making things like
`PythonCallableSource.of("math.sqrt")` or
`PythonCallableSource.of("scipy.special.ellipj")` work with minimal overhead,
and reduce the amount of overhead required for each calling SDK (go,
typescript, ...) by keeping the representation a simple string.
--
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]
