Query iterator remove throws UnsupportedOperation

Current implementaton is misleading, it remove from a temporary internal list, not from the collection. Simply not supporting the optional operation makes more sense.

Also, current implementation has a major bug. NoSuchElementException on no hasNext should have been IllegalStateException when next as not been called piror, and remove should be on currentIndex - 1.

Author: wjobin

src/com/microsoft/azure/documentdb/QueryIterable.java

@@ -117,14 +117,11 @@ public T next() {
             }
 
             /**
-             * Removes the current value.
+             * Remove not supported.
              */
             @Override
             public void remove() {
-                if (!hasNext()) throw new NoSuchElementException();
-                if (currentIndex < items.size()) {
-                    items.remove(currentIndex);
-                }
+                throw new UnsupportedOperationException("remove");
             }
 
         };