Skip to content

Commit

Permalink
fix: Use dateutil parser to parse materialization times (#2464)
Browse files Browse the repository at this point in the history
Signed-off-by: Achal Shah <[email protected]>
  • Loading branch information
achals authored Mar 29, 2022
1 parent ff7c7fa commit 6c55e49
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions sdk/python/feast/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
import pkg_resources
import yaml
from colorama import Fore, Style
from dateutil import parser

from feast import flags, flags_helper, utils
from feast.constants import DEFAULT_FEATURE_TRANSFORMATION_SERVER_PORT
Expand All @@ -41,7 +42,6 @@
)

_logger = logging.getLogger(__name__)
DATETIME_ISO = "%Y-%m-%dT%H:%M:%s"


class NoOptionDefaultFormat(click.Command):
Expand Down Expand Up @@ -501,8 +501,8 @@ def materialize_command(
store = FeatureStore(repo_path=str(repo))
store.materialize(
feature_views=None if not views else views,
start_date=utils.make_tzaware(datetime.fromisoformat(start_ts)),
end_date=utils.make_tzaware(datetime.fromisoformat(end_ts)),
start_date=utils.make_tzaware(parser.parse(start_ts)),
end_date=utils.make_tzaware(parser.parse(end_ts)),
)


Expand Down

0 comments on commit 6c55e49

Please sign in to comment.