fdb-record-layer icon indicating copy to clipboard operation
fdb-record-layer copied to clipboard

Resolve #1733 MatchCandidate type fields are generated with nondeterministic order

Open hatyo opened this issue 2 years ago • 7 comments

When generating the Type information for a MatchCandidate we use this RecordMetaData::getFieldDescriptorMap which returns a HashMap of fields that is translated later to a Record type.

This could cause a problem when attempting to reference some of these fields by ordinal (OrdinalFieldValue) as fields could be enlisted differently between a MatchCandidate and a flown type of another operator.

hatyo avatar Jun 15 '22 08:06 hatyo

Result of fdb-record-layer-pr-proto2 on Linux CentOS 7

  • Commit ID: 5718175ad0c2c5157a84fd511fe969eee4ec3a2c
  • Duration 0:15:23
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Logs (available for 30 days)

foundationdb-ci avatar Jun 15 '22 08:06 foundationdb-ci

Result of fdb-record-layer-pr-proto3 on Linux CentOS 7

  • Commit ID: 5718175ad0c2c5157a84fd511fe969eee4ec3a2c
  • Duration 0:16:05
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Logs (available for 30 days)

foundationdb-ci avatar Jun 15 '22 08:06 foundationdb-ci

Result of fdb-record-layer-pr-proto2 on Linux CentOS 7

  • Commit ID: 378c46c29056554c72201d40b67e7c4198a87a33
  • Duration 0:15:06
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Logs (available for 30 days)

foundationdb-ci avatar Jun 15 '22 11:06 foundationdb-ci

Result of fdb-record-layer-pr-proto3 on Linux CentOS 7

  • Commit ID: 378c46c29056554c72201d40b67e7c4198a87a33
  • Duration 0:15:34
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Logs (available for 30 days)

foundationdb-ci avatar Jun 15 '22 11:06 foundationdb-ci

Result of fdb-record-layer-pr-proto2 on Linux CentOS 7

  • Commit ID: 3f96d6feafd529562326d3b8f42e7c65930cf4f6
  • Duration 0:06:55
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest sonarqube -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Logs (available for 30 days)

foundationdb-ci avatar Jun 15 '22 15:06 foundationdb-ci

Result of fdb-record-layer-pr-proto3 on Linux CentOS 7

  • Commit ID: 3f96d6feafd529562326d3b8f42e7c65930cf4f6
  • Duration 0:07:02
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest sonarqube -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Logs (available for 30 days)

foundationdb-ci avatar Jun 15 '22 15:06 foundationdb-ci