Is there a way I can throw some error from jxpath when this situation arises instead of getting stuck in a never ending loop?
On Sun, 24 Sep, 2023, 22:54 Matt Benson, <gudnabr...@gmail.com> wrote: > Well, to be clear, you can have linked nodes. But if N had both `next` and > `previous` members is where you'd run into trouble. > > On Sun, Sep 24, 2023 at 12:22 PM Matt Benson <gudnabr...@gmail.com> wrote: > > > If you're doing a search down the tree you'd need some way to keep JXPath > > from traversing these relationships, yes. > > > > On Sun, Sep 24, 2023 at 11:59 AM Debraj Manna <subharaj.ma...@gmail.com> > > wrote: > > > >> Are you saying that in JxPath we cannot use something like below? > >> > >> class A { > >> int n; > >> A next; > >> } > >> > >> On Sun, Sep 24, 2023 at 8:43 PM Matt Benson <mben...@apache.org> wrote: > >> > >> > Is it possible the proto message (I'm not familiar with this API) is > >> built > >> > with internal recursive references, i.e. some child has a property > that > >> > points, possibly indirectly, to its parent? That would be the most > >> probable > >> > explanation, particularly as you say feeding jxpath the known absolute > >> path > >> > works. > >> > > >> > Matt > >> > > >> > On Sun, Sep 24, 2023, 9:44 AM Debraj Manna <subharaj.ma...@gmail.com> > >> > wrote: > >> > > >> > > It looks like it is getting stuck in some infinite loop. The thread > >> stack > >> > > looks like below after ~ 8 hours from the start. > >> > > > >> > > main Runnable CPU usage on sample: 979ms > >> > > jdk.internal.reflect.GeneratedMethodAccessor640.invoke() > >> > > > >> > > > >> > > > >> > > >> > jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > >> > > java.lang.reflect.Method.invoke(Method.java:568) > >> > > > >> org.apache.commons.jxpath.util.ValueUtils.getValue(ValueUtils.java:367) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.model.beans.BeanPropertyPointer.getBaseValue(BeanPropertyPointer.java:120) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.model.beans.BeanPropertyPointer.getImmediateNode(BeanPropertyPointer.java:149) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.model.beans.PropertyPointer.getImmediateValuePointer(PropertyPointer.java:161) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.model.NodePointer.getValuePointer(NodePointer.java:297) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.model.beans.PropertyIterator.getNodePointer(PropertyIterator.java:121) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.axes.DescendantContext.nextNode(DescendantContext.java:115) > >> > > > >> org.apache.commons.jxpath.ri.EvalContext.nextSet(EvalContext.java:349) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.axes.ChildContext.getSingleNodePointer(ChildContext.java:70) > >> > > > >> org.apache.commons.jxpath.ri.compiler.Path.searchForPath(Path.java:201) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.compiler.Path.getSingleNodePointerForSteps(Path.java:176) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.compiler.LocationPath.computeValue(LocationPath.java:87) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.JXPathContextReferenceImpl.getValue(JXPathContextReferenceImpl.java:353) > >> > > > >> > > > >> > > > >> > > >> > org.apache.commons.jxpath.ri.JXPathContextReferenceImpl.getValue(JXPathContextReferenceImpl.java:313) > >> > > > >> > > > >> > > > >> > > >> > com.spotnana.webhooks.isos.PNRListBuilderTest.testJxPathCtx(PNRListBuilderTest.java:366) > >> > > > >> > > On Sun, Sep 24, 2023 at 1:24 AM Debraj Manna < > >> subharaj.ma...@gmail.com> > >> > > wrote: > >> > > > >> > > > I am trying to search using JxPath on a proto message, My proto > >> message > >> > > is > >> > > > placed here > >> > > > < > >> https://gist.github.com/debraj-manna/4a8ded5d79dc9aee8d4c03d460327284 > >> > >. > >> > > > > >> > > > val protoStr = > >> > > > > getStringFromFile("classpath:BookingEnginePnrUpdateEventProto.txt"); > >> > > > val msgBld = BookingEnginePNRUpdateEvent.newBuilder(); > >> > > > TextFormat.getParser().merge(protoStr, msgBld); > >> > > > val msgCtx = JXPathContext.newContext(msgBld.build()); > >> > > > System.out.println(msgCtx.getValue("//employeeId")); > >> > > > > >> > > > I am observing that msgCtx.getValue("//employeeId") is taking a > >> lot of > >> > > > time. Thread dumps at various points are placed in the below gist. > >> > > > > >> > > > > >> https://gist.github.com/debraj-manna/47852fa190075abdacd527969f355cde > >> > > > > >> > > > If I specify the entire path like > >> > > > > >> > > > >> > > >> > msgCtx.getValue("/metadata/bookerInfo/traveler/userBusinessInfo/employeeId"). > >> > > > It is working fine. > >> > > > > >> > > > Can someone let me know what is going wrong? Is this expected? > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > >> > > >> > > >