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 <buyin...@gmail.com> wrote:

> Steven,
>
>     Can you add a regression test before you closing that issue?
>     (Typically, fixing ASTERIXDB-xxxx 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&focusedCommentId=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 <prest...@apache.org>
>> Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
>> Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
>>
>>
>> > 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)
>>
>
>

Reply via email to