这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@nizar-m
Copy link
Contributor

@nizar-m nizar-m commented Mar 14, 2019

Description

Allows following types of queries

query q {
  bar: foo (where: { a : { _contained_in : [1,2,3,4,5,6,7,8]  }} ) {
    a
  }

  baz: foo (where: { a : { _contains : [1,2,3,4] }} ) {
    a
  } 
}

And the following mutations with array

mutation m {
  insert_foo ( objects: [ {a: [1,3,5,7,9] } ]){
    affected_rows
    returning {
      a 
      id
    }
  }
}

Affected components

  • Server
  • Tests

Related Issues

#348
#349

Solution and Design

Steps to test and verify

Limitations, known bugs & workarounds

@nizar-m nizar-m requested a review from 0x777 as a code owner March 14, 2019 01:06
@nizar-m nizar-m requested a review from rakeshkky March 14, 2019 01:13
@hasura-bot
Copy link
Contributor

Review app for commit afae0f0 deployed to Heroku: https://hge-ci-pull-1767.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1767-afae0f0

@netlify
Copy link

netlify bot commented Mar 22, 2019

Deploy preview for hasura-docs ready!

Built with commit cb93ebc

https://deploy-preview-1767--hasura-docs.netlify.com

@hasura-bot
Copy link
Contributor

Review app for commit cb93ebc deployed to Heroku: https://hge-ci-pull-1767.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1767-cb93ebc

@shahidhk shahidhk added the c/server Related to server label Mar 27, 2019
@shahidhk
Copy link
Member

@nizar-m Please add the right merge label.

@nizar-m nizar-m added the s/do-not-merge Do not merge this pull request to master label Apr 3, 2019
@dsandip dsandip added this to the release-candidates milestone Apr 15, 2019
@rakeshkky
Copy link
Member

closing in favour of #2243

@rakeshkky rakeshkky closed this May 21, 2019
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-1767.herokuapp.com is deleted

hasura-bot pushed a commit that referenced this pull request Mar 21, 2025
<!-- The PR description should answer 2 important questions: -->

### What
Adds tests for `case` and `cast` in the full SQL pushdown path.

<!-- What is this PR trying to accomplish (and why, if it's not
obvious)? -->

<!-- Consider: do we need to add a changelog entry? -->

<!-- Does this PR introduce new validation that might break old builds?
-->

<!-- Consider: do we need to put new checks behind a flag? -->

### How

<!-- How is it trying to accomplish it (what are the implementation
steps)? -->

V3_GIT_ORIGIN_REV_ID: 7fc824958bb094059ba2b8b49afc9fe9f9324e49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/server Related to server s/do-not-merge Do not merge this pull request to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants