Community
Participate
Working Groups
There is a method on IQuery called getID which is supposed to return the ID of the query. The InstallableUnitQuery overrides this method and provides different semantics. With the p2ql work, I'm not sure the getID method is needed on IQuery anymore.
I would be happy to see that gone, or at least not marked API anymore.
(In reply to comment #1) > I would be happy to see that gone, or at least not marked API anymore. So would I. I'm still at a position where I think all queries should be able to present themselves as expressions if possible. And I still believe it is possible for a majority of our queries. With that in place, there is no need to do introspection of the queries to find out what they are all about and hence, no need for the methods getID() and getProperty(String). I suggest we get rid of both.
Created attachment 158072 [details] patch It's a pretty easy fix since nobody actually uses these methods. This patch removes the methods Removes the tests assoicated with them Removes the methods from sublcasses Removes the QueryHelper (and the package it was in) Fixes up the manifests
releaed to head.