[GraphQL] Add a new multiGetObjects query on Query.

[stefan-mysten]

Description

This PR adds a new query for fetching multiple objects by their ids and versions.

Test plan

How did you test the new or updated feature?

---

Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

• Protocol:
• Nodes (Validators and Full nodes):
• Indexer:
• JSON-RPC:
• GraphQL: added a multiGetObjects top level query, which will replace the objectFilter in objects.
• CLI:
• Rust SDK:
• REST API:

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sui-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 20, 2024 8:56pm

3 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
multisig-toolkit	⬜️ Ignored (Inspect)	Visit Preview		Dec 20, 2024 8:56pm
sui-kiosk	⬜️ Ignored (Inspect)	Visit Preview		Dec 20, 2024 8:56pm
sui-typescript-docs	⬜️ Ignored (Inspect)	Visit Preview		Dec 20, 2024 8:56pm

[amnn]

Requesting changes for the query limit checker stuff mainly!

[amnn]

Just the changes and tests related to query limits checker left, everything else is looking good here!

[amnn]

Looks good! Can you also add some E2E tests for the happy path (not hitting the limit) before landing?

[amnn]

Suggested change

	let objects = data
	.into_iter()
	.filter_map(|(lookup_key, bcs)| {
	Object::new_serialized(
	lookup_key.id,
	lookup_key.version,
	bcs,
	checkpoint_viewed_at,
	lookup_key.version,
	)
	})
	.collect::<Vec<_>>();
	let objects: Vec<_> = data
	.into_iter()
	.filter_map(|(lookup_key, bcs)| {
	Object::new_serialized(
	lookup_key.id,
	lookup_key.version,
	bcs,
	checkpoint_viewed_at,
	lookup_key.version,
	)
	})
	.collect();

[amnn]

The name of this constant seems to imply that it's specific to multiGetObject but it's not/shouldn't be -- all multiGets now need to use keys as their parameter name for holding the keys.

Suggested change

	let keys = f.node.get_argument(MULTI_GET_OBJECT_KEYS);
	let keys = f.node.get_argument(MULTI_GET_KEYS);

[amnn]

Suggested change

	let objects = data
	.into_iter()
	.filter_map(|(lookup_key, bcs)| {
	Object::new_serialized(
	lookup_key.id,
	lookup_key.version,
	bcs,
	checkpoint_viewed_at,
	lookup_key.version,
	)
	})
	.collect::<Vec<_>>();
	let objects: Vec<_> = data
	.into_iter()
	.filter_map(|(lookup_key, bcs)| {
	Object::new_serialized(
	lookup_key.id,
	lookup_key.version,
	bcs,
	checkpoint_viewed_at,
	lookup_key.version,
	)
	})
	.collect();