On Fri, Jan 1, 2010 at 2:20 PM,  <markrmil...@apache.org> wrote:
> ==============================================================================
> --- lucene/solr/branches/cloud/src/webapp/web/admin/zookeeper.jsp (original)
> +++ lucene/solr/branches/cloud/src/webapp/web/admin/zookeeper.jsp Fri Jan  1 
> 19:20:26 2010
> @@ -391,6 +391,11 @@
>           try {
>             str = new String(data, "UTF-8");
>           } catch (UnsupportedEncodingException e) {
> +            // nocommit : does this make sense? isn't UTF-8
> +            // gauranteed to be supported? I don't think this is thrown
> +            // because its not UTF-8 is it? The results are unspecified
> +            // when the bytes are not properly encoded.

Hmmm, yeah, we need a better way to detect malformed UTF-8 to fall
back to binary....
if (str.indexOf(utf_replacement_string) >= 0) ?

-Yonik
http://www.lucidimagination.com

Reply via email to