/*
 * Licensed to CRATE Technology GmbH ("Crate") under one or more contributor
 * license agreements.  See the NOTICE file distributed with this work for
 * additional information regarding copyright ownership.  Crate licenses
 * this file to you under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.  You may
 * obtain a copy of the License at
 *
 *   http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
 * License for the specific language governing permissions and limitations
 * under the License.
 *
 * However, if you have executed another commercial license agreement
 * with Crate these terms will supersede the license and you may use the
 * software solely pursuant to the terms of the relevant commercial agreement.
 */

package io.crate.operator.operations.collect;

import javax.annotation.Nullable;

public class ModuloBucketingIterator extends BucketingIterator {

    public ModuloBucketingIterator(int numBuckets, Iterable<Object[]> rowIterable) {
        super(numBuckets, rowIterable);
    }

    /**
     * get bucket number by doing modulo hashcode of first row-element
     */
    @Override
    protected int getBucket(@Nullable Object[] row) {
        if (row == null || row.length == 0) {
            return 0;
        } else {
            return row[0].hashCode() % this.numBuckets;
        }
    }
}

@@ -1553,16 +1553,25 @@
  return 
+Math.abs(
 row%5B0%5D.h
@@ -1579,16 +1579,17 @@
 shCode()
+)
  %25 this.

fix bucket generation in ModuloBucketingIterator