aino-io / agent-java Goto Github PK

Even if a size threshold of one is specified in agent configuration, agent sometimes sends batches of multiple transaction entries.

This is cumbersome in e.g. mocking scenarios.

Currently (1.1.4) the Java agent doesn't support environment specific values in the config xml-file. It would be useful to be able to use environment specific values to configure e.g. the aino API URI or the API key. Environment specific values could be inserted to the config file using e.g. the ${} notation.

java.util.ConcurrentModificationException
at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:859)
at java.util.ArrayList$Itr.next(ArrayList.java:831)
at io.aino.agents.core.TransactionDataBuffer.addTransaction(TransactionDataBuffer.java:56)
at io.aino.agents.core.Agent.addTransaction(Agent.java:112)
at io.aino.agents.wso2.mediator.AinoMediator.processTransaction(AinoMediator.java:263)
at io.aino.agents.wso2.mediator.AinoMediator.mediate(AinoMediator.java:198)
at org.apache.synapse.mediators.AbstractListMediator.mediate(AbstractListMediator.java:77)
at org.apache.synapse.mediators.AbstractListMediator.mediate(AbstractListMediator.java:47)
at org.apache.synapse.mediators.base.SequenceMediator.mediate(SequenceMediator.java:131)
at org.apache.synapse.mediators.eip.Target.mediate(Target.java:106)
at org.apache.synapse.mediators.eip.splitter.IterateMediator.mediate(IterateMediator.java:146)
at org.apache.synapse.mediators.AbstractListMediator.mediate(AbstractListMediator.java:77)
at org.apache.synapse.mediators.AbstractListMediator.mediate(AbstractListMediator.java:47)
at org.apache.synapse.mediators.filters.FilterMediator.mediate(FilterMediator.java:160)
at org.apache.synapse.mediators.AbstractListMediator.mediate(AbstractListMediator.java:77)
at org.apache.synapse.mediators.AbstractListMediator.mediate(AbstractListMediator.java:47)
at org.apache.synapse.mediators.base.SequenceMediator.mediate(SequenceMediator.java:131)
at org.apache.synapse.core.axis2.ProxyServiceMessageReceiver.receive(ProxyServiceMessageReceiver.java:166)
at org.apache.axis2.engine.AxisEngine.receive(AxisEngine.java:180)
at org.apache.axis2.transport.base.AbstractTransportListener.handleIncomingMessage(AbstractTransportListener.java:328)
at org.apache.synapse.transport.vfs.VFSTransportListener.processFile(VFSTransportListener.java:597)
at org.apache.synapse.transport.vfs.VFSTransportListener.scanFileOrDirectory(VFSTransportListener.java:328)
at org.apache.synapse.transport.vfs.VFSTransportListener.poll(VFSTransportListener.java:158)
at org.apache.synapse.transport.vfs.VFSTransportListener.poll(VFSTransportListener.java:107)
at org.apache.axis2.transport.base.AbstractPollingTransportListener$1$1.run(AbstractPollingTransportListener.java:67)
at org.apache.axis2.transport.base.threads.NativeWorkerPool$1.run(NativeWorkerPool.java:172)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)

Explain meaning of interval, sizeThreshold and gzipEnabled

When stopping a sender, the sender should wait until the transaction buffer is empty before cancelling the sender.

Currently on agent shutdown some data might not get sent if there are transactions in the buffer when agent shutdown is issued.

element should be optional, and agent should use reasonable default values, e.g.

• interval = 5000
• sizeThreshold = 30
• gzipEnabled = true

Sender class needs more comprehensive unit tests

Add a common parent class to all Aino Agent exceptions

Caused by: org.xml.sax.SAXParseException; lineNumber: 6; columnNumber: 43; cvc-complex-type.2.4.d: Invalid content was found starting with element 'proxy'. No child element is expected at this point.

Agent API should have a method for shutting down the agent (=terminating sender threads and other clean up).

Background: when using the agent in a batch processing application, the application does not shut down correctly since agent sender threads remain running on the background (and it would be nice to be able to gracefully shut down the agent in a application server environment as well)

Proxy -element should be before send -element