AnandInguva commented on code in PR #28205: URL: https://github.com/apache/beam/pull/28205#discussion_r1309329280
########## sdks/python/apache_beam/ml/transforms/tft.py: ########## @@ -68,6 +68,15 @@ _LOGGER = logging.getLogger(__name__) +def warn_about_consistent_artifact_output( + cls_name: str, artifacts_name: List[str]): + _LOGGER.warning( + "The outputs of the get_artifacts method of class %s" % cls_name + + " are computed over the " + "entire dataset. For each transformed element, the corresponding " + "artifacts %s will be the same." % artifacts_name) Review Comment: Okay. it makes sense to add it to the FAQ/troubleshooting section. I can add an info level log saying that the artifacts are added to the output. -- 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]
