Skip to content
Snippets Groups Projects

adapt graphql request to Hasura engine

Merged poka requested to merge adapt-to-hasura into master
4 unresolved threads

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
288 288 (
289 289 info.cert_issued
290 290 .into_iter()
291 .map(|i| i.receiver.name.to_string())
291 .map(|i| i.receiver.unwrap().name.to_string())
292 292 .collect(),
293 293 info.cert_received
294 294 .into_iter()
295 .map(|i| i.issuer.name.to_string())
295 .map(|i| i.issuer.unwrap().name.to_string())
  • poka
    poka @pokapow started a thread on the diff
  • 57 63 }
    58 64
    59 65 query LatestBlock {
    60 blocks(limit: 1, orderBy: height_DESC) {
    66 block(limit: 1, orderBy: { height: DESC }) {
    61 67 height
    62 68 hash
    63 69 }
    64 70 }
    65 71
    66 72 query BlockByNumber($number: Int!) {
    67 blocks(where: { height_eq: $number }) {
    73 block(where: { height: { _eq: $number } }) {
    68 74 height
    69 75 hash
    • Author Owner

      Actuellement bug sur le parsing de ces hash, au format Bytea en DB, donc avec \\ au lieu de 0x en début de string pour de l'hexa. En cours de résolution côté indexer.

    • Il faut juste oublier les deux premiers caractères et interpréter la suite comme de l'hexadécimal, pas besoin de corriger ça côté indexeur, ça n'a aucun impact.

    • Please register or sign in to reply
  • Author Owner

    Nécessite quelques adaptation côté indexer avant de merger.

  • Hugo Trentesaux added 1 commit

    added 1 commit

    • 14693a0c - example on how to deserialize

    Compare with previous version

  • Je t'ai mis un exemple de déserialisation au cas où.

  • poka added 2 commits

    added 2 commits

    • 06807a47 - 2 macros to rule them all
    • edf8dabb - Ignore non_camel_case_types in vscode stage

    Compare with previous version

    • Author Owner

      J'ai vue, du coup ce n'est pas juste un exemple, tout fonctionne >< J'ai ajouté un ignore dans le .vscode car il doit y avoir un bug de rust-analyzer qui nous dis non_camel_case_types alors que c'est déjà ignoré sur la ligne... J'ai essayé d'ignorer pour l'ensemble du module, ça ne change rien.

      J'ai encore du mal en Rust avec le fait de ne pas savoir où se passe une erreur, l'analyzer t'indique un endroit, mais ça peut se passer dans l'implémentation de cet endroit, bien plus loins ...

    • Je pense que c'est du côté des macro. Le problème camel case est à l'intérieur du code généré.

      Et il faut pas garder ce que tu as mis pour débugger : macro_rules! graphql_query_debug

      [edit] c'est le diff gitlab qui est pas lisible

      Edited by Hugo Trentesaux
    • J'ai mis une macro de plus, et je suggère de ne mettre Debug que quand on en a besoin. Par contre ton indexeur semble à la bourre :

      image

    • Author Owner

      Ah oui en effet:

      duniter-squid-processor-1     | {"level":2,"time":1709683182174,"ns":"sqd:processor","msg":"435160 / 435160, rate: 0 blocks/sec, mapping: 86 blocks/sec, 259 items/sec, eta: 0s"}
      duniter-squid-processor-1     | {"level":2,"time":1709683188188,"ns":"sqd:processor","msg":"435161 / 435161, rate: 0 blocks/sec, mapping: 91 blocks/sec, 274 items/sec, eta: 0s"}
      duniter-squid-processor-1     | {"level":2,"time":1709683194091,"ns":"sqd:processor","msg":"435162 / 435162, rate: 0 blocks/sec, mapping: 90 blocks/sec, 270 items/sec, eta: 0s"}
      duniter-squid-processor-1     | {"level":2,"time":1709683200099,"ns":"sqd:processor","msg":"435163 / 435163, rate: 0 blocks/sec, mapping: 90 blocks/sec, 269 items/sec, eta: 0s"}
      duniter-squid-processor-1     | {"level":2,"time":1709683608909,"ns":"sqd:processor","msg":"435172 / 435172, rate: 0 blocks/sec, mapping: 138 blocks/sec, 415 items/sec, eta: 0s"}
      duniter-squid-processor-1     | {"level":5,"time":1709683609149,"ns":"sqd:processor","err":{"generatedMessage":true,"code":"ERR_ASSERTION","actual":false,"expected":true,"operator":"==","stack":"AssertionError [ERR_ASSERTION]: The expression evaluated to a falsy value:\n\n  (0, assert_1.default)(head.height >= this.chain[0].height)\n\n    at HotProcessor.moveToBlocks (/squid/node_modules/@subsquid/util-internal-ingest-tools/lib/hot.js:60:30)\n    at HotProcessor.goto (/squid/node_modules/@subsquid/util-internal-ingest-tools/lib/hot.js:38:24)\n    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n    at async RpcDataSource.handleNewHeads (/squid/node_modules/@subsquid/substrate-data-raw/lib/datasource.js:213:24)\n    at async RpcDataSource.subscription (/squid/node_modules/@subsquid/substrate-data-raw/lib/datasource.js:198:25)\n    at async RpcDataSource.processHotBlocks (/squid/node_modules/@subsquid/substrate-data-raw/lib/datasource.js:113:13)"}}

      J'ouvre une issue à ce sujet. Super pratique cette commande gcli, je l'utilisais déjà pour tester le code en local avec noeud et indexer local.

      Edited by poka
    • Please register or sign in to reply
  • poka marked this merge request as ready

    marked this merge request as ready

  • poka approved this merge request

    approved this merge request

  • Hugo Trentesaux added 1 commit

    added 1 commit

    Compare with previous version

  • poka added 1 commit

    added 1 commit

    Compare with previous version

  • Hugo Trentesaux added 1 commit

    added 1 commit

    Compare with previous version

  • poka mentioned in commit c6d37c20

    mentioned in commit c6d37c20

  • Please register or sign in to reply
    Loading