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

Conversation

@praveenweb
Copy link
Member

@praveenweb praveenweb commented Dec 4, 2018

Description

What component does this PR affect?

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Requires changes from other components? If yes, please mark the components:

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Related Issue

Solution and Design

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Docs update
  • Community content

Checklist:

  • I have read the contributing guide and my code conforms to the guidelines.
  • This change requires a change in the documentation.
  • I have updated the documentation accordingly.
  • I have added required tests.

@hasura-bot
Copy link
Contributor

Review app for commit dcc19c5 deployed to Heroku: https://hge-ci-pull-1165.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1165-dcc19c5

@shahidhk shahidhk requested a review from karthikvt26 December 5, 2018 06:53
karthikvt26
karthikvt26 previously approved these changes Dec 5, 2018
@shahidhk
Copy link
Member

shahidhk commented Dec 5, 2018

@praveenweb Lot of changes in package lock where https is replaced by http. Intentional?

@shahidhk shahidhk added the c/console Related to console label Dec 5, 2018
@praveenweb
Copy link
Member Author

@shahidhk - Seems to be an issue - npm/npm#20106

Anyways, have modified package-lock to have https now.

@shahidhk shahidhk merged commit 65d476c into hasura:master Dec 5, 2018
@hasura-bot
Copy link
Contributor

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

@praveenweb praveenweb deleted the platform-remote-schemas branch May 21, 2019 06:58
hasura-bot pushed a commit that referenced this pull request Sep 30, 2024
<!-- The PR description should answer 2 important questions: -->

### What

<!-- 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? -->
Add traces to `graphql-ws` crate. Since websockets are implemented by
running multiple parallel tasks, we are using span links to establish
relation between traces emitted by those tasks.

Refactor by moving code into new modules and updating function names and
types.

### How
- Define a new `in_span_async_with_link()` in tracing-util to enable
creating a span with a link.
- Wrap spans around necessary code sections with attaching required
attributes.
- Small no-op refactor in `common.rs` test file.

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

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

Labels

c/console Related to console

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants