[ 
https://issues.apache.org/jira/browse/SOLR-8522?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15104043#comment-15104043
 ] 

Arcadius Ahouansou edited comment on SOLR-8522 at 1/18/16 1:44 AM:
-------------------------------------------------------------------

Thank you very much [~noble.paul]ul] for taking the time to look into this

{quote}
The solr node string may not always be an IP address. It could be something 
like {[host:port}} . So IP address needs a lookup
{quote}
You are right about this. I was not aware of this.
Turned out that a user could start Solr with -Dhost=someHostName.
Doing the lookup as suggested is quite simple. However, 1 host could have 
multiple public and private IPs. We could pick the first public one or 
something...

This led me to start contemplating the idea of a more generic snitch that will 
deal with host names as well as IPs like
{{192.168.1.2 -> host_1=2, host_2=1, host_3=168, host_4=192}}
and
{{serv1.dc1.london.uk.apache.org -> host_1=org, host_2=apache, host_3=uk, 
host_4=london, host_5=dc1, host_6=serv1}} 

Any comment about this?


{quote}Let's start from least significant to most significant{quote}
Yes, makes sense


{quote}Do not blindly add a tag . Add if it is only requested{quote}

The current implementation adds only the tags that are requested.
The one that are not requested are not added to the response.
This is tested in 
- {{testGetTagsWithEmptyIPv4RequestedTag()}} where no tag is requested -> none 
returned, and
 
- {{testGetTagsWithIPv4RequestedTags_ip_2_ip_4()}} where only 2 tags are 
requested leading to only 2 out of 4 being returned 


Please let me know about the idea of a more generic snitch that could handle 
host names as well.

Many thanks



was (Author: arcadius):
Thank you very much [~noble.paul]ul] for taking the time to look into this

{quote}
The solr node string may not always be an IP address. It could be something 
like {[host:port}} . So IP address needs a lookup
{quote}
You are right about this. I was not aware of this.
Turned out that a user could start Solr with -Dhost=someHostName.
Doing the lookup as suggested is quite simple. However, 1 host could have 
multiple public and private IPs. We could pick the first public one or 
something...

This led me to start contemplating the idea of a more generic snitch like
{{192.168.1.2 -> host_1=2, host_2=1, host_3=168, host_4=192}}
and
{{serv1.dc1.london.uk.apache.org -> host_1=org, host_2=apache, host_3=uk, 
host_4=london, host_5=dc1, host_6=serv1}} 

Any comment about this?


{quote}Let's start from least significant to most significant{quote}
Yes, makes sense


{quote}Do not blindly add a tag . Add if it is only requested{quote}

The current implementation adds only the tags that are requested.
The one that are not requested are not added to the response.
This is tested in 
- {{testGetTagsWithEmptyIPv4RequestedTag()}} where no tag is requested -> none 
returned, and
 
- {{testGetTagsWithIPv4RequestedTags_ip_2_ip_4()}} where only 2 tags are 
requested leading to only 2 out of 4 being returned 


Please let me know about the idea of a more generic snitch that could handle 
host names as well.

Many thanks


> ImplicitSnitch to support IPv4 based tags
> -----------------------------------------
>
>                 Key: SOLR-8522
>                 URL: https://issues.apache.org/jira/browse/SOLR-8522
>             Project: Solr
>          Issue Type: Improvement
>          Components: SolrCloud
>    Affects Versions: 5.4
>            Reporter: Arcadius Ahouansou
>            Assignee: Noble Paul
>            Priority: Minor
>         Attachments: SOLR-8522.patch
>
>
> This is a description from [~noble.paul]'s comment on SOLR-8146
> Lets assume a Solr node IPv4 address is 192.93.255.255 .
> This is about enhancing the current {{ImplicitSnitch}}  to support IP based 
> tags like:
> - {{ip_1 = 192}}
> - {{ip_2 = 93}}
> - {{ip_3 = 255}}
> - {{ip_4 = 255}}
> Note that IPv6 support will be implemented by a separate ticket



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to