Summary
Paper is interested in sampling subgraphs for training GNNs on large input graphs, rather than doing gradient decent steps on the whole graph at every training step. They write a general training pipeline that can be applied on most message passing GNNs (with AGGREGATE and COMBINE), including GraphSAGE, GCN, and others. They theoretically show that, under some assumptions, the parameters recovered by subgraph training are not too far away form the parameters of full-graph training.
Weaknesses
# Experiments
The paper is lacking on experiments -- there are only two datasets: pubmed (small size) and ogb-mag (medium size). It is nice to show some wins on more datasets, especially larger ones. I also suggest to remove the word "very" for "very large citation networks", on 2nd page 3rd paragraph.
In fact, the SIGN model (also designed to scale GNNs, by computing pretraining ($Z = AAX$) then training on rows $Z$ without ever looking at adjacency matrix $A$ ever again during training, i.e., the graph is only used for untrainable pre-processing then usual SGD training proceeds.
# Comparison with earlier work
The work on GTTF (Marowitz et al, ICLR 2021) is very similar: it develops a general algorithm that can be specialized for training GNNs and skip-gram models (like deepwalk) by sampling, they also prove that the parameters learned with sampling are equivalent to ones learned on full graph, under some assumptions (the actual assumptions and proofs in GTTF differ).
It would be nice if this paper brings in something new: if it is the analysis, name the paper appropriately. Otherwise, it feels like just another paper that is doing sampling (like GraphSAGE, GTTF, etc)
# Uninterpretable math
* The notation is not standard. E.g., instead of $READOUT(h_v, v\in V)$, it could be $READOUT([h_v]_{v \in V})$ -- As-is, it looks like a function accepting 2 arguments: a feature vector and a boolean (is $v$ a member of $V$).
* Further, the expression for $\mathcal{P}_n$ at end of page 5 is hard to decipher
* In section 3.2, there are 3 conflicting definition of $\mathcal{v}_g$. It says $\mathcal{v}_g : G \rightarrow R^+$ but then invokes it on same line as $v_g(G, v)$
* Having READOUT function (last line of Alg.1) is ambiguous: how are the sampled nodes any special to give a signal to the entire graph?
* Is $\mu$ a measure or an instance? In definition 4.1, it overloads it to be both
* What is $B$ above Eq 6? Is it the same as defined later in Sec 4.2?
# Minor Improvements
* In Section 2: GNNAutoScale is **not** sampling-based GNN. On the other hand, it is based on historical embeddings.
* Below Equation 2, you say "simple aggregation". Why not be explicit and define "simple" right here?