[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, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[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, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[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.



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org