Use JSON.MGET for repo.fetch multiple ids

[chiawendt]

Currently repo.fetch(ids) translates to awaiting multiple JSON.GET. Performance might be improved by replacing JSON.GET with JSON.MGET.

redis-om-node/lib/repository/repository.ts

Lines 355 to 363 in 6f08d02

	private async readEntitiesFromJson(ids: string[]): Promise<T[]> {
	return Promise.all(
	ids.map(async (entityId): Promise<T> => {
	const keyName = this.makeKey(entityId)
	const jsonData = await this.client.jsonget(keyName) ?? {}
	const entityData = fromRedisJson(this.#schema, jsonData)
	return {...entityData, [EntityId]: entityId, [EntityKeyName]: keyName} as T
	}))
	}

[guyroyse]

That's a good callout. I'm working on Redis OM 0.5.0 right now and will add this to my backlog.

[Mathieu-Lala]

Hi there, how about also adding JSON.MSET ?