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.


---

Reply via email to