Re: [jira] [Commented] (ASTERIXDB-1587) upsert runtime assumes that the datatype comes from the same dataverse as the dataset

2016-08-17 Thread Yingyi Bu
>> But in this case it seemed like the test was unnecessary, although I do
have tests on my BAD branch that require this change.

It seems useful to me as it will protect the system against future changes.
Could you submit a separate new change which only has a test case?
Thanks a lot!

Best,
Yingyi


On Wed, Aug 17, 2016 at 6:19 PM, Steven Jacobs  wrote:

> I already pushed the change :(
> But in this case it seemed like the test was unnecessary, although I do
> have tests on my BAD branch that require this change.
> Steven
>
> On Wed, Aug 17, 2016 at 6:15 PM, Yingyi Bu  wrote:
>
>> Steven,
>>
>> Can you add a regression test before you closing that issue?
>> (Typically, fixing ASTERIXDB- should have a test case in the
>> change.)
>> Thanks!
>>
>> Best,
>> Yingyi
>>
>> On Wed, Aug 17, 2016 at 6:05 PM, ASF subversion and git services (JIRA) <
>> j...@apache.org> wrote:
>>
>>>
>>> [ https://issues.apache.org/jira/browse/ASTERIXDB-1587?page=co
>>> m.atlassian.jira.plugin.system.issuetabpanels:comment-tabpan
>>> el=15425696#comment-15425696 ]
>>>
>>> ASF subversion and git services commented on ASTERIXDB-1587:
>>> 
>>>
>>> Commit ddc760f59b906e3124a5da7d73fa7d3575fd79b1 in asterixdb's branch
>>> refs/heads/master from [~sjaco002]
>>> [ https://git-wip-us.apache.org/repos/asf?p=asterixdb.git;h=ddc760f ]
>>>
>>> ASTERIXDB-1587 Made upsert look in the correct dataverse for datatypes
>>>
>>> Change-Id: Iccf04c2210a1535cd7de6eeea88dfefec9ba206a
>>> Reviewed-on: https://asterix-gerrit.ics.uci.edu/1085
>>> Reviewed-by: Preston Carman 
>>> Sonar-Qube: Jenkins 
>>> Tested-by: Jenkins 
>>>
>>>
>>> > upsert runtime assumes that the datatype comes from the same dataverse
>>> as the dataset
>>> > 
>>> -
>>> >
>>> > Key: ASTERIXDB-1587
>>> > URL: https://issues.apache.org/jira
>>> /browse/ASTERIXDB-1587
>>> > Project: Apache AsterixDB
>>> >  Issue Type: Bug
>>> >Reporter: Steven Jacobs
>>> >Assignee: Steven Jacobs
>>> >
>>> > It throws an error if they aren't from the same dataverse. It should
>>> work instead.
>>>
>>>
>>>
>>> --
>>> This message was sent by Atlassian JIRA
>>> (v6.3.4#6332)
>>>
>>
>>
>


Re: [jira] [Commented] (ASTERIXDB-1587) upsert runtime assumes that the datatype comes from the same dataverse as the dataset

2016-08-17 Thread Yingyi Bu
Steven,

Can you add a regression test before you closing that issue?
(Typically, fixing ASTERIXDB- should have a test case in the
change.)
Thanks!

Best,
Yingyi

On Wed, Aug 17, 2016 at 6:05 PM, ASF subversion and git services (JIRA) <
j...@apache.org> wrote:

>
> [ https://issues.apache.org/jira/browse/ASTERIXDB-1587?page=
> com.atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel=15425696#comment-15425696 ]
>
> ASF subversion and git services commented on ASTERIXDB-1587:
> 
>
> Commit ddc760f59b906e3124a5da7d73fa7d3575fd79b1 in asterixdb's branch
> refs/heads/master from [~sjaco002]
> [ https://git-wip-us.apache.org/repos/asf?p=asterixdb.git;h=ddc760f ]
>
> ASTERIXDB-1587 Made upsert look in the correct dataverse for datatypes
>
> Change-Id: Iccf04c2210a1535cd7de6eeea88dfefec9ba206a
> Reviewed-on: https://asterix-gerrit.ics.uci.edu/1085
> Reviewed-by: Preston Carman 
> Sonar-Qube: Jenkins 
> Tested-by: Jenkins 
>
>
> > upsert runtime assumes that the datatype comes from the same dataverse
> as the dataset
> > 
> -
> >
> > Key: ASTERIXDB-1587
> > URL: https://issues.apache.org/jira
> /browse/ASTERIXDB-1587
> > Project: Apache AsterixDB
> >  Issue Type: Bug
> >Reporter: Steven Jacobs
> >Assignee: Steven Jacobs
> >
> > It throws an error if they aren't from the same dataverse. It should
> work instead.
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.3.4#6332)
>