Working with Observable.create

gistfile1.txt

Hi,

I'm pretty new to RxJava and trying to do something quite simple but getting weird behaviour...

Please take a look at:
Tester#processMessages
XMLParser#parse

I create an Observable and from the subscriber I do `subscriber.onComplete();` but it doesn't invoke the .doOnCompleted(),
on the other hand, if I invoke `subscriber.onNext(e);` then the doOnNext(...) is triggered!

I attached a zip with a small reproducer.

I would appreciate any help ...

Thank you!

Tester.java

import java.io.*;

import javax.xml.parsers.*;
import javax.xml.transform.*;
import javax.xml.transform.dom.*;
import javax.xml.transform.stream.*;

import org.w3c.dom.*;

import io.vertx.core.*;
import io.vertx.core.net.*;
import io.vertx.rxjava.core.AbstractVerticle;
import io.vertx.rxjava.core.buffer.*;
import io.vertx.rxjava.core.net.NetClient;
import io.vertx.rxjava.core.net.NetSocket;
import io.vertx.rxjava.core.streams.*;



public class Tester extends AbstractVerticle {
	
	NetSocket socket = null;
	
	public Tester() {
		super.vertx = io.vertx.rxjava.core.Vertx.vertx(new VertxOptions()
			.setBlockedThreadCheckInterval(5*60*1000)
		);
	}

	public static void main(String[] args) {
			
		new Tester().connect("bgblitz.no-ip.org", 12344);
		
		try {
			Thread.currentThread().join();
		} catch (InterruptedException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}
	
	public void connect(String host, int port) {
		
		NetClient client = vertx.createNetClient(new NetClientOptions()
			.setConnectTimeout(10000)
			.setReconnectAttempts(10)
			.setReconnectInterval(500)
		);
		
		client.connect(port, host, socketObs -> {
			if(socketObs.succeeded()) {
				
				System.out.println("Connected!");
				
				this.socket = socketObs.result();
				
				Pump pump = Pump.pump(socket, socket);
				pump.start();
				
				processMessages(socket);
			
				socket.closeHandler(event -> {
					System.out.println("CloseHandler triggered; Event=" + event);
				});
				
				socket.exceptionHandler(e -> {
					System.out.println("ExceptionHandler triggered; Event=" + e);
				});
				
				sendSetupRequest();
			}
		});
	}
	
	
	private void processMessages(ReadStream<Buffer> f) {
		
		XMLParser parser = new XMLParser();
		
		f.toObservable()
		.map(buf -> (io.vertx.core.buffer.Buffer)buf.getDelegate())
		.flatMap(parser::parse)
		.doOnNext(e -> { System.out.println("Next: " + e); } )
		.doOnCompleted(() -> { System.out.println("COMPLETED!"); })
		.subscribe();
	}
	
	private void sendSetupRequest() {
		
		try {

            DocumentBuilderFactory fact = DocumentBuilderFactory.newInstance();
            DocumentBuilder build = fact.newDocumentBuilder();
            Document doc = build.newDocument();
            
            Element setupReq = doc.createElement("SetupRequest");
            doc.appendChild(setupReq);
            
            setupReq.setAttribute("ply", "2");
            setupReq.setAttribute("threads", "2");
            setupReq.setAttribute("trace", "setup_log.txt");
            
            TransformerFactory tFact = TransformerFactory.newInstance();
            Transformer trans = tFact.newTransformer();
            trans.setOutputProperty(OutputKeys.INDENT, "yes");

            StringWriter writer = new StringWriter();
            StreamResult result = new StreamResult(writer);
            DOMSource source = new DOMSource(doc);
            trans.transform(source, result);
            
            
            socket.write(writer.toString());

        } catch (TransformerException | ParserConfigurationException e) {
            System.out.println("Error outputting document");
        }
    }
}

XMLParser.java

import javax.xml.stream.*;

import com.fasterxml.aalto.*;
import com.fasterxml.aalto.stax.*;

import io.vertx.core.buffer.*;

public class XMLParser {
	private AsyncXMLStreamReader<AsyncByteArrayFeeder> reader = new InputFactoryImpl().createAsyncForByteArray();
	
	public rx.Observable<XMLStreamEvent> parse(Buffer data) {
		
		byte[] bytes = data.getBytes();
		final int bufSize = data.length();
		
		try {
			reader.getInputFeeder().feedInput(bytes, 0, bufSize);
		
		} catch (XMLStreamException e) {
			return rx.Observable.error(e);
		}

		return rx.Observable.create(subscriber -> {
			
			// For some reason, the 'doOnComplete(..)' isn't triggered !
			subscriber.onCompleted();
			
			// If I uncomment these lines and comment the above line --> the doOnNext(..) gets triggered!
//			XMLStreamEvent e = new XMLStreamEvent(7, 0, reader);
//			subscriber.onNext(e);
		});
	}
}

Missing the class XMLStreamEvent it seams not to be part of any imported packages.
It would be nice if you could provide it. Make clear from which import it is coming from.

And it would be nice to know which version of Java libraries (jars) are being used.