Github user tliron commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/207#discussion_r153014611
--- Diff: aria/parser/reading/yaml.py ---
@@ -82,7 +84,11 @@ def read(self):
# see issue here:
#
https://bitbucket.org/ruamel/yaml/issues/61/roundtriploader-causes-exceptions-with
#yaml_loader = yaml.RoundTripLoader(data)
- yaml_loader = yaml.SafeLoader(data)
+ try:
+ # Faster C-based loader, might not be available on all
platforms
+ yaml_loader = yaml.CSafeLoader(data)
+ except BaseException:
--- End diff --
I think I'm sure ... I want the failover to always work. Even if
CSafeLoader does exist but fails somehow internally, we should still make an
effort to run.
---