KevinGG commented on a change in pull request #11020: [BEAM-7926] Update Data Visualization URL: https://github.com/apache/beam/pull/11020#discussion_r387330714
########## File path: sdks/python/apache_beam/runners/interactive/options/interactive_options.py ########## @@ -24,6 +24,8 @@ from __future__ import absolute_import +from dateutil import tz Review comment: The user can use both. Here internally we use dateutil.tz get the local timezone info. Externally, the user can use `pytz.timezone` or `dateutil.tz.gettz` because the `to_tz` just needs to be a subclass of datetime.tzinfo. Added the comments in the exposed API. ---------------------------------------------------------------- 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
