ibzib commented on pull request #14008: URL: https://github.com/apache/beam/pull/14008#issuecomment-780912101
> I actually like everything here, this change will enable me to submit a compliance test covering BeamJavaUdfCalcRule inside Google. How can we test `BeamJavaUdfCalcRule` if it never matches anything though? For that reason I think it's better to go with #14009 and then remove `BeamJavaUdfCalcRule` from the default rules. That way all you have to do to test `BeamJavaUdfCalcRule` is add the rule. ---------------------------------------------------------------- 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]
