[GitHub] [beam] TheNeuralBit commented on pull request #12426: [BEAM-7996] Add support for MapType and Nulls in container types for Python RowCoder

2020-08-05 Thread GitBox
TheNeuralBit commented on pull request #12426: URL: https://github.com/apache/beam/pull/12426#issuecomment-669604112 Run Java PreCommit This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] TheNeuralBit commented on pull request #12426: [BEAM-7996] Add support for MapType and Nulls in container types for Python RowCoder

2020-08-05 Thread GitBox
TheNeuralBit commented on pull request #12426: URL: https://github.com/apache/beam/pull/12426#issuecomment-669588208 Run Java PreCommit This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] TheNeuralBit commented on pull request #12426: [BEAM-7996] Add support for MapType and Nulls in container types for Python RowCoder

2020-07-30 Thread GitBox
TheNeuralBit commented on pull request #12426: URL: https://github.com/apache/beam/pull/12426#issuecomment-666590444 This is relevant for BEAM-10571. Some existing payloads use `List>` fields, which we could replace with Map when using schemas to encode the payload.