Noted.


On Thu, May 2, 2013 at 5:24 PM, Jeff Kolesky <notifications@github.com>wrote:

> In
> kiji-mapreduce/src/main/java/org/kiji/mapreduce/framework/SmartKijiTableInputFormat.java:
>
> > +
> > +      // Get the start and limit keys. Ensure they exist.
> > +      String postHashStartKeyEncoded = conf.get(KIJI_POST_HASH_START_KEY);
> > +      if (Strings.isNullOrEmpty(postHashStartKeyEncoded)) {
> > +        throw new IOException(String.format("Must define configuration property [%s]",
> > +                                            KIJI_POST_HASH_START_KEY));
> > +      }
> > +      // Get our desired start key not including the hash. If all of the bytes in this key
> > +      // are "full", then we can't use it as a prefix.
> > +      byte[] postHashStartKey = Base64.decodeBase64(postHashStartKeyEncoded);
> > +      byte[] unincrementableKey = new byte[postHashStartKey.length];
> > +      Arrays.fill(unincrementableKey, (byte)-1);
> > +      if (Bytes.equals(postHashStartKey, unincrementableKey)
> > +          || Bytes.equals(postHashStartKey, new byte[postHashStartKey.length])) {
> > +        // This could probably use a better error message, and better explication overall.
> > +        throw new IOException(String.format("Invalid value [%s] for configuration propert [%s]: "
>
> I'm guessing "propert" is not the word you meant to use.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/opower/kiji-mapreduce/pull/4/files#r4067207>
> .
>