Skip to content
This repository was archived by the owner on Dec 13, 2023. It is now read-only.

Conversation

hkernbach
Copy link
Member

This PR adds the documentation for the experimental feature:
Programmable Pregel Algorithms

@hkernbach hkernbach added this to the 3.8 milestone Sep 9, 2020
Copy link
Member

@mchacki mchacki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general it looks very good already.

I have added some points.

Lars Maier and others added 2 commits December 3, 2020 13:34
@maierlars
Copy link

I addressed some issues. I would like to fix those issues that require a change in the code base first in the code and then come back to the docs.

Copy link
Member

@goedderz goedderz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the last part of my review.

maierlars and others added 3 commits December 4, 2020 16:40
Copy link
Member

@goedderz goedderz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, some of the comments do not actually concern the documentation itself, but seemed easier to discuss in this context.

algorithmic action takes place. An AIR program is represented with the Arango
Intermediate Representation (AIR).

## Arango Intermediate Representation
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having used this document for a while, I think it might be good to move the AIR part of the documentation out of this document into a separate one.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Simran-B what do you think?

@goedderz goedderz requested a review from mchacki February 25, 2021 12:45
@goedderz goedderz removed the 9 WIP label Feb 25, 2021
@Simran-B Simran-B merged commit d566aca into main Feb 25, 2021
@Simran-B Simran-B deleted the feature/experimental-custom-programmable-pregel-algos branch February 25, 2021 18:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

5 participants