Yes, the parser should just care about syntax.
Semantic checks should be done in the translator or later.
Cheers,
Till
On 29 Sep 2015, at 14:59, Yingyi Bu wrote:
In ConnectedFeedStatement, a similar piece of code has been commented
out.
IMO, the AQL parser should just get an AST from a query, but not
access the
metadata nor do any real work..
Best,
Yingyi
On Tue, Sep 29, 2015 at 2:46 PM, Ian Maxon <[email protected]> wrote:
I always wondered where that plan's input came from in the CC logs.
It gets
generated during a connect statement as well.
On Tue, Sep 29, 2015 at 2:04 PM, Mike Carey <[email protected]>
wrote:
I wasn't aware of that statement...!
On Sep 29, 2015 12:17 PM, "Yingyi Bu" <[email protected]> wrote:
All right, I will open an issue for that.
Thanks!
Best,
Yingyi
On Tue, Sep 29, 2015 at 12:11 PM, abdullah alamoudi <
[email protected]>
wrote:
I am not aware of any special reason and it definitely looks a bit
too
hackish to me.
I would say that it needs to be fixed but I don't think it is a
priority
at this point. Anyway, it is a private command that is not exposed
to
the
end user.
I would like to know if there is a reason as well.
~Abdullah.
Amoudi, Abdullah.
On Tue, Sep 29, 2015 at 9:53 PM, Yingyi Bu <[email protected]>
wrote:
Does anyone know why SubscribeFeedStatement in asterix-aql needs
to
access the MetadataManager to form yet-another AQL insert query
inside
it
and hand that to the AQLParser again?
It seems a bit hackish to me. Is there a particular reason that
it
must
be done this way?
Thanks!
Best,
Yingyi