gregfelice commented on PR #2468: URL: https://github.com/apache/age/pull/2468#issuecomment-5073484210
Checking in — this has been quiet since Monday. @eeshwarg nothing needed from you yet beyond a preference: land this as-is with a benchmark at realistic edge-label counts, or fix the plan structure here along the lines in #2479. Either works for me and I'll re-review promptly. @jrgemignani @MuhammadTahaNaveed — a call from one of you on which path to take would unblock this. John, this also answers your Jul 13 performance question: the alternation path resolves to `_ag_label_edge`, so the scan expands across every edge label table in the graph. The cost scales with the total number of edge labels present, not with the number listed in the pattern — so it's invisible on a small test graph and grows with the schema. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
