HIVE自定义UDAF中将部分结果汇总时需要注意的一个地方

最近有些了一个UDAF,在调试时,遇到了一些奇怪的现象,明明逻辑正确,却总是得不到预期的结果。而且测试环境正确,但是放到生产环境的不正确。通过对应中间变量日志,看到以下数据:

以上是map的日志输出,可以看到,两次输入的value数组,在第四行的oprations(中间缓存变量)时,变成了两个一模一样的数组。这是一个典型的引用变量问题。观察代码如下:

private ArrayList<ArrayList<String>> operations=new ArrayList<ArrayList<String>>() ;

//每次对一个新值进行聚集计算都会调用iterate方法,相当于mapper。//将输入参数存入ArrayList
        public boolean iterate(ArrayList<String> value){
        	System.out.println("----------------------1iterate_value:"+value);
        	//value的格式为:rn,opt,end_ts
        	if (value == null || value.size()==0){
        		return true;
        	}
        	System.out.println("----------------------3iterate.value.xx:"+value);
        	operations.add(value);
        	System.out.println("----------------------8iterate.operations:"+operations);
			return true;
        }

可以看到,value和operations都是数组列表,value由框架传入,operations的赋值是通过operations.add(value)这句完成,框架在传递value时,应该是重用了上一次的列表对象,而operations的add方法,实际上是只保存了被添加对象的地址,所以下一次迭代(下一行数据)时,value中的值被替换时,operations中所有的元素都跟着变了,因为operations中引用的都是同一个内存地址。

解决的方法很简单,就是对框架传入的对象,使用clone()方法:



		//每次对一个新值进行聚集计算都会调用iterate方法,相当于mapper。//将输入参数存入ArrayList
        public boolean iterate(ArrayList<String> value){
        	System.out.println("----------------------1iterate_value:"+value);

        	//value的格式为:rn,opt,end_ts
        	if (value == null || value.size()==0){
        		return true;
        	}
        	System.out.println("----------------------3iterate.value.xx:"+value);
 
        	
        	operations.add((ArrayList<String>) value.clone()); //将结果存储中间变量,这里必须用clone方法,或者new一个新的对象,否则会有bug
        	System.out.println("----------------------8iterate.operations:"+operations);
			return true;
        	
        }

猜你喜欢

转载自blog.csdn.net/cakecc2008/article/details/105240309