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

cname missing from logger-follow results in rocksdb #2868

Closed
darkfrog26 opened this issue Jul 25, 2017 · 2 comments
Closed

cname missing from logger-follow results in rocksdb #2868

darkfrog26 opened this issue Jul 25, 2017 · 2 comments
Assignees
Milestone

Comments

@darkfrog26
Copy link

darkfrog26 commented Jul 25, 2017

my environment running ArangoDB

I'm using the latest ArangoDB of the respective release series:

  • 3.2

Mode:

  • Single-Server

Storage-Engine:

  • rocksdb

On this operating system:

  • Linux
    • Ubuntu .deb

After upgrading to ArangoDB 3.2 and switching to RocksDB querying from logger-follow no longer returns cname for collection related data. Specifically, I'm testing with a DocumentUpsert and getting the following response:

{"cid":"11257","data":{"_id":"test/11270","_key":"11270","_rev":"_VVWPGUG---","age":28,"name":"Jane Doe"},"database":"1","tick":"76764","tid":"0","type":2300}

Everything looks right except the lack of cname. Even the documentation doesn't seem to recognize that this has changed: https://docs.arangodb.com/3.2/HTTP/Replications/ReplicationLogger.html

@jsteemann
Copy link
Contributor

The "cname" attribute should be back for RocksDB starting in 3.2.1

@jsteemann jsteemann added the 2 Fixed Resolution label Jul 28, 2017
@jsteemann jsteemann added this to the 3.2.1 milestone Jul 28, 2017
@jsteemann jsteemann self-assigned this Jul 28, 2017
ObiWahn added a commit that referenced this issue Aug 3, 2017
…ture/js_to_cpp_transaction_handler

* 'devel' of https://github.com/arangodb/arangodb:
  fixed some issues detected by coverity scan (#2915)
  remove dependency on MMFiles features from non-MMFiles files (#2925)
  Converted rest handler for explain from JS to C++. (#2907)
  added startsBefore() for ApplicationFeature (#2913)
  Converted a portion of the admin routing API from JS to C++ (#2919)
  Bug fix/predictable results data modifcation multiple fors (#2921)
  Feature/cpp aql char length (#2883)
  slightly move responsibility for recovery (#2922)
  Feature/issue 387 cluster index estimates (#2866)
  Converted endpoint handler from JS to C++ (#2905)
  switch to trusty
  fixed issue #2876 (#2896)
  fix a race on shutdown (#2897)
  fixed issue #2868: cname missing from logger-follow results in rocksdb (#2901)
  make the V8 feature depend on the authentication feature (#2902)
  change "parameters" to "bindVars" (#2772)
  Do not allow replication to create/drop collections (#2898)
  don't mask errors with fake OOM messages (#2872)
@jsteemann
Copy link
Contributor

3.2.1 has been released and is available for download.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants