dalibo / pev2

Postgres Explain Visualizer 2
https://explain.dalibo.com
PostgreSQL License
2.55k stars 124 forks source link

Update help-service.ts #570

Closed meob closed 1 year ago

meob commented 1 year ago

Adding GATHER and GATHER MERGE node description

pgiraud commented 1 year ago

Thanks for your contribution. May I suggest to rephrase it a bit?

pgiraud commented 1 year ago

Here is what my colleagues proposed:

« Gather » reads the results of the parallel workers, in an undefined order.
« Gather Merge » reads the results of the parallel workers, preserving any ordering.

If you agree with these new statements, would you be kind to update your PR? Thanks.

meob commented 1 year ago

Here is what my colleagues proposed:

« Gather » reads the results of the parallel workers, in an undefined order.
« Gather Merge » reads the results of the parallel workers, preserving any ordering.

If you agree with these new statements, would you be kind to update your PR? Thanks.

Of course: better than mine... sorry for the delay I had to change my primary email