Avoid N+1 selects from user table when packing many entities

[zyoshoka]

Summary

エンティティを packMany する際、単純に pack に map している .*EntityService が複数あります。

misskey/packages/backend/src/core/entities/AbuseUserReportEntityService.ts

Lines 53 to 58 in eaadd64

	@bindThis
	public packMany(
	reports: any[],
	) {
	return Promise.all(reports.map(x => this.pack(x)));
	}

この際 pack 内で UserEntityService.pack が呼び出されている場合、キャッシュが効かない最悪の場合には packMany に渡される配列の要素数回 user テーブルからの SELECT クエリが走ることになります。

misskey/packages/backend/src/core/entities/AbuseUserReportEntityService.ts

Lines 32 to 50 in eaadd64

	return await awaitAll({
	id: report.id,
	createdAt: this.idService.parse(report.id).date.toISOString(),
	comment: report.comment,
	resolved: report.resolved,
	reporterId: report.reporterId,
	targetUserId: report.targetUserId,
	assigneeId: report.assigneeId,
	reporter: this.userEntityService.pack(report.reporter ?? report.reporterId, null, {
	schema: 'UserDetailedNotMe',
	}),
	targetUser: this.userEntityService.pack(report.targetUser ?? report.targetUserId, null, {
	schema: 'UserDetailedNotMe',
	}),
	assignee: report.assigneeId ? this.userEntityService.pack(report.assignee ?? report.assigneeId, null, {
	schema: 'UserDetailedNotMe',
	}) : null,
	forwarded: report.forwarded,
	});

#13550 で UserEntityService.packMany が最適化されたので、このようなケースでは単純に map するのではなく、前段で packMany を呼び出して1回のクエリで済ませた上でそれを pack に渡すほうがより適切だと思われます。

Purpose

パフォーマンスの向上

Do you want to implement this feature yourself?

• Yes, I will implement this by myself and send a pull request