asterixdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann" <ti...@apache.org>
Subject Re: SubscribeFeedStatement
Date Tue, 29 Sep 2015 22:04:54 GMT
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 <imaxon@uci.edu> 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 <dtabass@gmail.com> 
>> wrote:
>>
>>> I wasn't aware of that statement...!
>>> On Sep 29, 2015 12:17 PM, "Yingyi Bu" <buyingyi@gmail.com> wrote:
>>>
>>>> All right, I will open an issue for that.
>>>> Thanks!
>>>>
>>>> Best,
>>>> Yingyi
>>>>
>>>> On Tue, Sep 29, 2015 at 12:11 PM, abdullah alamoudi <
>> bamousaa@gmail.com>
>>>> 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 <buyingyi@gmail.com>
>> 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
>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>

Mime
View raw message