Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stream Transaction HTTP docs #8833

Merged
merged 9 commits into from Apr 29, 2019
Merged

Stream Transaction HTTP docs #8833

merged 9 commits into from Apr 29, 2019

Conversation

graetzer
Copy link
Contributor

@graetzer graetzer commented Apr 23, 2019

@graetzer graetzer added this to the devel milestone Apr 23, 2019

### Stream Transactions

The [Stream Transactions](StreamTransaction.md) allows you to perform a multi-document transaction
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The [Stream Transactions](StreamTransaction.md) allows you to perform a multi-document transaction
[Stream Transactions](StreamTransaction.md) allow you to perform a multi-document transaction

### JavaScript Transactions

The [JS-Transaction API](JsTransaction.md) allows you to send the server
a dedicated peace of JavaScript code (i.e. a function), which will be executed transactionally.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
a dedicated peace of JavaScript code (i.e. a function), which will be executed transactionally.
a dedicated piece of JavaScript code (i.e. a function), which will be executed transactionally.

HTTP Interface for Stream Transactions
======================================

*Stream Transactions* allows you to perform a multi-document transaction
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
*Stream Transactions* allows you to perform a multi-document transaction
*Stream Transactions* allow you to perform a multi-document transaction

There is a new HTTP API for transactions. This API allows clients to add operations to a
transaction in a streaming fashion. A transaction can consist of a series of supported
transactional operations, followed by a commit or abort command.
This allows clients to construct larger transactions in a more efficent way then
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This allows clients to construct larger transactions in a more efficent way then
This allows clients to construct larger transactions in a more efficent way than

transaction in a streaming fashion. A transaction can consist of a series of supported
transactional operations, followed by a commit or abort command.
This allows clients to construct larger transactions in a more efficent way then
with the JavaScript based transactions.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
with the JavaScript based transactions.
with the JavaScript-based transactions.


Note that this requires client applications to abort transactions which are no
longer necessary. Otherwise resources and locks acquired by the transactions
will hang around until the server decides to garbage collect them.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
will hang around until the server decides to garbage collect them.
will hang around until the server decides to garbage-collect them.

@@ -58,6 +58,8 @@ Please refer to [Locking and Isolation](LockingAndIsolation.md) for more details
In Clusters
-----------

TODO: fix this section up
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be removed before we merged this PR.

@jsteemann jsteemann merged commit b110f5f into devel Apr 29, 2019
ObiWahn added a commit that referenced this pull request Apr 30, 2019
…-fix/foxx-security-windows-symlinks

* 'devel' of https://github.com/arangodb/arangodb:
  fix a few issues in BackupNoAuthSysTest (#8868)
  track number of failed requests that want to bump the plan version (#8867)
  next hack attempt (#8802)
  Improve estimator apply algo + test (#8871)
  Stream Transaction HTTP docs (#8833)
  Improve path finder tests (#8849)
@fceller fceller deleted the feature/stream-trx-docs branch July 30, 2019 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants