ibzib commented on pull request #13733:
URL: https://github.com/apache/beam/pull/13733#issuecomment-758842693


   Thanks for contributing. It looks like you will need to add a comma after 
`FN_JSON_VALUE` and remove the comma after `FN_SHA512_STRING` so that the list 
compiles correctly.
   
   I will leave the rest of the review to @amaliujia.


----------------------------------------------------------------
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:
[email protected]


Reply via email to