Apache Answer项目中短ID导致的关联问题查询错误分析

Apache Answer是一个开源的问答系统平台，在最新发布的1.4.2版本中出现了一个与短ID功能相关的数据库查询错误。本文将深入分析该问题的技术背景、产生原因以及解决方案。

问题背景

在Apache Answer系统中，管理员可以在SEO设置中启用"短ID"功能，这是一种更友好的URL表现形式。然而，当启用短ID后，系统在创建新问题时会在日志中记录数据库错误。

错误现象

系统日志中会出现如下错误信息：

ERROR question_common/question.go:712 get linked question ids error code: 500, reason: base.database_error, message: , error: pq: invalid input syntax for type bigint: "D112"

这表明系统尝试将一个短ID字符串（如"D112"）直接作为bigint类型插入PostgreSQL数据库，导致了类型不匹配的错误。

技术分析

数据库表结构

系统使用question_link表存储问题之间的关联关系，其关键字段定义如下：

CREATE TABLE public.question_link (
    id bigint NOT NULL,
    from_question_id bigint DEFAULT 0 NOT NULL,
    to_question_id bigint DEFAULT 0 NOT NULL,
    ...
);

代码逻辑缺陷

问题出现在question_common/question.go文件的第710行附近。系统在处理关联问题时，直接将短ID形式的questionID传递给数据库查询，而没有先将其转换为实际的数字ID。

正确的做法应该是使用uid.DeShortID()函数先将短ID转换为数字ID，再进行数据库操作。这个转换过程在系统的其他部分已经被正确实现，但在关联问题查询这一特定场景中被遗漏了。

解决方案

修复方案相对简单，只需在调用GetLinkedQuestionIDs方法前对questionID进行转换：

linkedQuestionIDs, err := qs.questionRepo.GetLinkedQuestionIDs(ctx, uid.DeShortID(questionID), entity.QuestionLinkStatusDeleted)

影响评估

虽然这个错误会导致日志中出现错误信息，但实际功能似乎仍然能够正常工作。这是因为：

1. 系统在其他地方正确处理了短ID转换
2. 关联关系最终仍被正确存储到数据库中
3. 错误发生在查询而非写入操作

不过，长期来看，这种错误可能会影响系统稳定性，特别是在高负载情况下，应该及时修复。

最佳实践建议

对于类似系统，在处理ID转换时应注意：

1. 统一ID处理逻辑，避免部分转换、部分不转换的情况
2. 在数据库接口层进行严格的类型检查
3. 对可能出现的转换错误进行适当的错误处理和日志记录
4. 编写单元测试覆盖各种ID格式的使用场景

通过这次问题的分析，我们可以看到在系统设计中保持数据转换逻辑的一致性是多么重要。这也提醒开发者在实现类似功能时，需要全面考虑各种使用场景下的数据流处理。