TheNeuralBit commented on issue #10055: [BEAM-8603] Add Python SqlTransform URL: https://github.com/apache/beam/pull/10055#issuecomment-613720045 It looks like PreCommit failures are just flakes or persistent failures at this point. I think I've addressed all of the PR comments, I just need an answer on two things: - @robertwb: Is it acceptable to skip sql_test for the default runner for now, since it doesn't support java_artifacts? - @ihji: Are the groovy changes ok given that they're outside of the `.each` block?
---------------------------------------------------------------- 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] With regards, Apache Git Services
