[GraphBolt] items are not shuffled across the whole set if num_workers>0

[Rhett-Ying]

🔨Work Item

IMPORTANT:

• This template is only for dev team to track project progress. For feature request or bug report, please use the corresponding issue templates.
• DO NOT create a new work item if the purpose is to fix an existing issue or feature request. We will directly use the issue in the project tracker.

Project tracker: DGL Project Tracker

Description

split item before shuffle results in significant accuracy drop. We should shuffle across the whole set first, then split items among workers. It's worth checking if torch.DL shuffle in this way.

buffer_size of ItemShufflerAndBatcher could always be len(item_set) to simplify the code logic.

Depending work items or issues

[Rhett-Ying]

For ItemSampler , it's fixed in #6982. One side effect of this fix is all workers are using same seed generator for DistributedItemSampler's shuffle.