Uh oh!
There was an error while loading. Please reload this page.
Add extra_returning_fields in apply - #767
Conversation
b3be96a to
24aa060Compare24aa060 to
c224408Comparewwwjfy
commented
Jun 20, 2021
Sorry for the late. I'm thinking of a parameter to return/reload every column instead of specified ones only. It saves a get call, which is probably being used in such case now. Another nitpicky point is the usage of tuple in this PR. As I understand, list is more suitable here, semantically. Quoting Python doc: |
kigawas
commented
Jun 20, 2021
Yes. Tuple should be replaced by Sequence in Python, which sugguests an immutable sequence although you can still pass list.
It's certainly better to get all columns by default, but for now a transitional approach would be more realistic to avoid API change |
wwwjfy
commented
Jun 20, 2021
I don't mean that big a change, but just an additional parameter like |
Closes#730