benWize commented on pull request #15562: URL: https://github.com/apache/beam/pull/15562#issuecomment-925962428
Hi, @ajamato would you help me to review this? I create a new class to override `flush` method in `MutationsBatcher` so I can get the rows response of `table.mutate_rows`. Each row has a google.rpc.status_pb2.Status for some of these codes https://cloud.google.com/bigtable/docs/status-codes is still missing how I can handle the grpc status codes because service_call_metrics is expecting int httpStatus or string status, but here the `grpc` status is an integer (0-14). https://github.com/apache/beam/blob/82f08b4dabc923151683f69cde71372755c7227c/sdks/python/apache_beam/io/gcp/bigtableio.py#L124 -- 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]
