Code

Add DataStreamDownload API#198

Code

Merged

corbin-phipps merged 46 commits intodevelopmicrosoft/netremote:developfrom feature/dataStreamingmicrosoft/netremote:feature/dataStreamingCopy head branch name to clipboardMar 1, 2024

+602-19

Commits

Commits on Feb 27, 2024

• Add DataStreamDownload API to proto files

  corbin-phipps

  committed
  e0b1bac

• Add basic API trace

  corbin-phipps

  committed
  3513304

• Merge pull request #182 from microsoft/user/corbinphipps/add-basic-tracing

  Show description for c61a7ce
  corbin-phipps

  authored
  c61a7ce

Commits on Feb 28, 2024

• Move away from new/delete

  corbin-phipps

  committed
  4f61279

• Merge pull request #184 from microsoft/user/corbinphipps/avoid-manual-memory-management

  Show description for edf8e5f
  corbin-phipps

  authored
  edf8e5f

• Merge branch 'feature/dataStreaming' into user/corbinphipps/add-download-api

  corbin-phipps

  committed
  d28a641

• Sample implementation from prototyping

  corbin-phipps

  committed
  fd36f99

• Improve enum value names; add unknown enum values

  corbin-phipps

  committed
  3c48ff9

• Improve API impl

  corbin-phipps

  committed
  e3b31bd

• Fill in OnCancel() callback

  corbin-phipps

  committed
  3f7398c

• Switch back to new/delete

  corbin-phipps

  committed
  68e4080

• Add ClientReadReactor impl

  corbin-phipps

  committed
  5a1f8c2

• Add basic unit test; delete this in OnDone

  corbin-phipps

  committed
  97e3b12

• Add missing break statements

  corbin-phipps

  committed
  e809317

• clang-format

  corbin-phipps

  committed
  ac74b9b

• Update comment

  corbin-phipps

  committed
  dfd38c3

• Merge pull request #185 from microsoft/user/corbinphipps/add-download-api

  Show description for c8ad32c
  corbin-phipps

  authored
  c8ad32c

Commits on Feb 29, 2024

• Simplify Properties oneof field names

  corbin-phipps

  committed
  2279f25

• const ref for string

  corbin-phipps

  committed
  27b20d8

• Add client reactor class member for timeout value

  corbin-phipps

  committed
  a3f2bec

• Re-order message fields so Status is first

  corbin-phipps

  committed
  992a635

• Allocate DataStreamReader on stack

  corbin-phipps

  committed
  4eec9e6

• Add chrono header

  corbin-phipps

  committed
  b9918e0

• Merge pull request #187 from microsoft/user/corbinphipps/download-api-improvements

  Show description for da6bc6b
  corbin-phipps

  authored
  da6bc6b

• Add unit test for continuous streaming (not passing yet, need to handle concurrency on server side)

  corbin-phipps

  committed
  2f95f76

• Add atomic bool for cancellation; add some logs

  corbin-phipps

  committed
  94c4145

• Address review comments: Update canceled spelling; extra logs

  corbin-phipps

  committed
  2ec99a0

• Use compare_exchange_strong

  corbin-phipps

  committed
  4aa65a6

Commits on Mar 1, 2024

• Merge pull request #191 from microsoft/user/corbinphipps/continuous-download-stream

  Show description for 53912d6
  corbin-phipps

  authored
  53912d6

• Merge branch 'develop' into user/corbinphipps/merge-develop

  corbin-phipps

  committed
  7a6603d

• Merge pull request #194 from microsoft/user/corbinphipps/merge-develop

  Show description for 533dd9c
  corbin-phipps

  authored
  533dd9c

• Add basic random data generator

  corbin-phipps

  committed
  6609de5

• Check data stream pattern

  corbin-phipps

  committed
  f7adbb4

• Use 32-bit version; use std::numeric_limits<uint8_t>::max; add const

  corbin-phipps

  committed
  93c0b67

• Merge pull request #195 from microsoft/user/corbinphipps/random-data-generator

  Show description for c5e4c2b
  corbin-phipps

  authored
  c5e4c2b

• Use function tracer with debug severity

  corbin-phipps

  committed
  adc8261

• Merge pull request #197 from microsoft/user/corbinphipps/use-function-tracer

  Show description for c00680f
  corbin-phipps

  authored
  c00680f

• Include headers explicitly.

  abeltrano

  committed
  ffd925f

• Drop args from FunctionTracer.

  abeltrano

  committed
  9d3fc40

• Style/formatting.

  abeltrano

  committed
  e697182

• const

  abeltrano

  committed
  ebb1799

• More clang-tidy.

  abeltrano

  committed
  92be9be

• Use uint32_t for distribution then cast to uint8_t

  corbin-phipps

  committed
  1927023

• Merge pull request #200 from microsoft/user/corbinphipps/fix-build-break

  Show description for d712312
  corbin-phipps

  authored
  d712312

• Merge pull request #199 from microsoft/clangtidy

  Show description for c64e9d5
  abeltrano

  authored
  c64e9d5

• Merge branch 'develop' into feature/dataStreaming

  abeltrano

  authored
  96b5d5e