Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

invalid literal for int() with base 10: '3-iap1' when refreshing druid metadata #872

Closed
jeffoxenberg opened this issue Aug 1, 2016 · 1 comment

Comments

@jeffoxenberg
Copy link

I'm trying to add a druid cluster to Caravel. I can add it OK, but the metadata refresh is failing with "Error while processing cluster '10.82.5.3' invalid literal for int() with base 10: '3-iap1'". My druid install is via Imply, and it seems that Caravel is barfing while trying to parse the version (0.8.3-iap1) because it isn't expecting the "-iap1".

After a cursory look at the code it may be around here: https://github.com/airbnb/caravel/blob/master/caravel/models.py#L1111 but i'm just guessing..

Ubuntu 14.04, Druid 0.8.3 (via Imply 1.1.0)

@jeffoxenberg
Copy link
Author

Now that I look at this more, the method I linked to above works OK to address this but version_higher() in the pip-installable package is older and does not address the trailing characters. I updated it to the latest on github and it's working OK, so i'm closing this.

zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 17, 2021
)

Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.0.11 to 2.0.17. **This update includes a security fix.**
- [Release notes](https://github.com/cure53/DOMPurify/releases)
- [Commits](cure53/DOMPurify@2.0.11...2.0.17)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
)

Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.0.11 to 2.0.17. **This update includes a security fix.**
- [Release notes](https://github.com/cure53/DOMPurify/releases)
- [Commits](cure53/DOMPurify@2.0.11...2.0.17)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
)

Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.0.11 to 2.0.17. **This update includes a security fix.**
- [Release notes](https://github.com/cure53/DOMPurify/releases)
- [Commits](cure53/DOMPurify@2.0.11...2.0.17)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
)

Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.0.11 to 2.0.17. **This update includes a security fix.**
- [Release notes](https://github.com/cure53/DOMPurify/releases)
- [Commits](cure53/DOMPurify@2.0.11...2.0.17)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant