search

pystorm / streamparse

Run Python in Apache Storm topologies. Pythonic API, CLI tooling, and a topology DSL.
http://streamparse.readthedocs.io/
Apache License 2.0
1.5k stars 218 forks source link

Regex Bug #468

Open dpelkmann opened 5 years ago

dpelkmann commented 5 years ago

Hello streamparse,

thanks for the hint (#303), that for a hdp cluster we have to insert the Hortonworks Storm Release Repo.

In my special case, I have a HDP Storm version which is not recognized by the regex expression in following function: https://github.com/Parsely/streamparse/blob/82199b1c409d07201b23e2d9d0f39b604ffec6a6/streamparse/util.py#L368-L384

My HDP Storm version is: 1.2.1.3.0.0.0-1634 and I got a version conflict because the hyphen was recognized in the configuration file but ignored in the above function. For me the following addition of the hyphen helped: 

pattern = r"Storm ([-0-9.]+)"

Best regards David

doshu commented 5 years ago

I am going to open the same issue. I confirm the bug and the proposed solution.