Review which data fields we're actually using

未关闭
  • 哪里不对？
#90 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 2 天内回复

还没有人认领这个 Issue。

评估

难度
5/5
预计耗时
一周以上
新手友好度
25/100
Issue 类型
重构
描述清晰度
需要澄清
活跃度
停滞
技术栈
graphql, rust
领域
backend-api-design

调研方向

首先检查 issue 中提到的模型结构体—RunMetadata、Attributes 和 Links—并追踪它们在查询处理和 mock server 测试中的使用情况。完成的标准是确定哪些字段未被使用或不可靠，决定哪些字段应保留或更改，并更新受影响的测试和 mock 响应。

由索引模型根据 Issue 内容生成。

描述

The model structs we currently have closely follow the data sent by tiled but much of it is not used or not reliable.

  • Search responses have a "meta" field with optional and arbitrary data
  • RunMetadata has a "start: Start" field mapping to the start document but this is never used in glazed (and can't be relied on to be anything more than an arbitrary json value).
  • Attributes has "access_blob", "specs",
  • Links contains a combination of all the link fields used by any of the types in tiled and might be better served by a hashmap
  • etc

It might be useful to remove the fields we're not using and replacing them if and when they're being used in glazed.

It would make testing a lot easier when we don't have to return a vast amount of unrelated data from mock servers to test query handling.

主要语言
Rust
星标
0
派生
2
平均合并
3 天 3 小时
30 天内合并 PR
20

环境准备

在 Codespaces 中打开

在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。

  • 提供 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 没有贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

DiamondLightSource/glazed 的其他 Issue

查看 DiamondLightSource/glazed 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。