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

Check that the user provided host parameter value is an IP address #49

Merged
merged 1 commit into from
Jul 12, 2019

Conversation

morazow
Copy link
Contributor

@morazow morazow commented Jul 10, 2019

This should help to fail-fast if users provide a domain name or non-ip value for the host parameter when using the Spark Exasol connector.

Fixes #43.

This should help to fail-fast if users provide a domain name or non-ip
value for the host parameter when using the Spark Exasol connector.
@morazow morazow requested review from 3cham and jpizagno July 10, 2019 14:10
Copy link
Contributor

@3cham 3cham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me :) well done @morazow

@@ -51,14 +53,25 @@ final case class ExasolConfiguration(

object ExasolConfiguration {

val IPv4_DIGITS: String = "(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this rule accepts 192.168.2.01 as valid address, I don't know if sub-connection could resolve this address (ping does). I'm currently having problem running it tests with my mac so I cannot verify it. Maybe could you please quickly check with our cluster @jpizagno ?

@morazow morazow merged commit c04da4f into master Jul 12, 2019
@morazow morazow deleted the task/#43-check-host-value branch July 15, 2019 08:20
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

Successfully merging this pull request may close these issues.

Check Host Name
2 participants