commit d98bc12cc1e888cbd8147516e4fa7639af419adf
author Bernd Mathiske <bernd@mesosphere.io> Mon Mar 10 13:14:44 2014 -0700
committer Niklas Q. Nielsen <niklas@mesosphere.io> Wed Apr 16 11:37:08 2014 -0700
tree c250448ca1f65d0815aaf28e2ae46f538c2e1aa2
parent 01326e62fa3f43c9527f69b7b149bf8941f0f770

Fixed JNI call for hasNext() in requestResources.

Apparently a typical copy-paste mistake of the programmer of this
code. hasNext() was called on ArrayList instead of on iterator.

Review: https://reviews.apache.org/r/18976

src/java/jni/org_apache_mesos_MesosSchedulerDriver.cpp

diff --git a/src/java/jni/org_apache_mesos_MesosSchedulerDriver.cpp b/src/java/jni/org_apache_mesos_MesosSchedulerDriver.cpp
index d2369ac..d0b7152 100644
--- a/src/java/jni/org_apache_mesos_MesosSchedulerDriver.cpp
+++ b/src/java/jni/org_apache_mesos_MesosSchedulerDriver.cpp
@@ -904,7 +904,7 @@
 
   jmethodID next = env->GetMethodID(clazz, "next", "()Ljava/lang/Object;");
 
-  while (env->CallBooleanMethod(jrequests, hasNext)) {
+  while (env->CallBooleanMethod(jiterator, hasNext)) {
     // Object task = iterator.next();
     jobject jrequest = env->CallObjectMethod(jiterator, next);
     const Request& request = construct<Request>(env, jrequest);