apple / foundationdb

FoundationDB - the open source, distributed, transactional key-value store
https://apple.github.io/foundationdb/
Apache License 2.0
14.19k stars 1.29k forks source link

Document format for SetSpanParent #11350

Closed gm42 closed 2 months ago

gm42 commented 2 months ago

SetSpanParent does not take a 16-bytes trace id as documentation mentions, but rather requires a specific serialized binary format (described in description field).

I would merge this to the release-7.3 release branch as well.

Code-Reviewer Section

The general pull request guidelines can be found here.

Please check each of the following things and check all boxes before accepting a PR.

For Release-Branches

If this PR is made against a release-branch, please also check the following:

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-clang-ide on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-clang on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-macos on macOS Ventura 13.x

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-clang-ide on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-clang on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr on Linux CentOS 7

jzhou77 commented 2 months ago

Looks like compilation failed:

[284/1833] cd /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/fdbclient && /usr/bin/mono /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/vexillographer.exe /codebuild/output/src4202394945/src/github.com/apple/foundationdb/fdbclient/vexillographer/fdb.options cpp /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/fdbclient/FDBOptions.g
FAILED: fdbclient/FDBOptions.g.h fdbclient/FDBOptions.g.cpp /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/fdbclient/FDBOptions.g.h /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/fdbclient/FDBOptions.g.cpp 
cd /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/fdbclient && /usr/bin/mono /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/vexillographer.exe /codebuild/output/src4202394945/src/github.com/apple/foundationdb/fdbclient/vexillographer/fdb.options cpp /codebuild/output/src4202394945/src/github.com/apple/foundationdb/build_output/fdbclient/FDBOptions.g

Unhandled Exception:
System.FormatException: Index (zero based) must be greater than or equal to zero and less than the size of the argument list.
  at System.Text.StringBuilder.AppendFormatHelper (System.IFormatProvider provider, System.String format, System.ParamsArray args) [0x000f9] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at System.String.FormatHelper (System.IFormatProvider provider, System.String format, System.ParamsArray args) [0x00023] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at System.String.Format (System.IFormatProvider provider, System.String format, System.Object arg0) [0x00008] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at System.IO.TextWriter.WriteLine (System.String format, System.Object arg0) [0x00007] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at vexillographer.cpp.writeCppInfo (System.IO.TextWriter outFile, vexillographer.Scope scope, System.Collections.Generic.IEnumerable`1[T] options) [0x00076] in <b399b67bcaa449f0b71171d5ab3c5236>:0 
  at vexillographer.cpp.writeFiles (System.String fileName, System.Collections.Generic.IEnumerable`1[T] options) [0x0018d] in <b399b67bcaa449f0b71171d5ab3c5236>:0 
  at vexillographer.vexillographer.Main (System.String[] args) [0x00082] in <b399b67bcaa449f0b71171d5ab3c5236>:0 
[ERROR] FATAL UNHANDLED EXCEPTION: System.FormatException: Index (zero based) must be greater than or equal to zero and less than the size of the argument list.
  at System.Text.StringBuilder.AppendFormatHelper (System.IFormatProvider provider, System.String format, System.ParamsArray args) [0x000f9] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at System.String.FormatHelper (System.IFormatProvider provider, System.String format, System.ParamsArray args) [0x00023] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at System.String.Format (System.IFormatProvider provider, System.String format, System.Object arg0) [0x00008] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at System.IO.TextWriter.WriteLine (System.String format, System.Object arg0) [0x00007] in <5dfd69ae4e3b402db546d8ded6fc755e>:0 
  at vexillographer.cpp.writeCppInfo (System.IO.TextWriter outFile, vexillographer.Scope scope, System.Collections.Generic.IEnumerable`1[T] options) [0x00076] in <b399b67bcaa449f0b71171d5ab3c5236>:0 
  at vexillographer.cpp.writeFiles (System.String fileName, System.Collections.Generic.IEnumerable`1[T] options) [0x0018d] in <b399b67bcaa449f0b71171d5ab3c5236>:0 
  at vexillographer.vexillographer.Main (System.String[] args) [0x00082] in <b399b67bcaa449f0b71171d5ab3c5236>:0 
gm42 commented 2 months ago

It looks like {...} is an expression used for formatting, will remove it.

Edit: done.

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-clang-ide on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-macos on macOS Ventura 13.x

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-clang on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr on Linux CentOS 7

foundationdb-ci commented 2 months ago

Result of foundationdb-pr-cluster-tests on Linux CentOS 7