Skip to content

Commit

Permalink
Merge pull request #70 from dmnbars/master
Browse files Browse the repository at this point in the history
remove unused properties
  • Loading branch information
jky-yy authored Dec 1, 2019
2 parents 8a55777 + b4a894a commit b5ed624
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
8 changes: 0 additions & 8 deletions src/Jaeger/Jaeger.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,24 +34,16 @@ class Jaeger implements Tracer{

private $scopeManager;

public static $handleProto = null;

public $spans = [];

public $tags = [];

public $process = null;

public $procesSize = 0;

public $serverName = '';

public $bufferSize = '';

public $processThrift = '';

public $spanThrifts = [];

public $propagator = null;

public function __construct($serverName = '', Reporter $reporter, Sampler $sampler,
Expand Down
2 changes: 0 additions & 2 deletions src/Jaeger/Transport/TransportUdp.php
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,6 @@ public function append(Jaeger $jaeger)
continue;
}

$jaeger->spanThrifts[] = $spanThrift;

if ($this->bufferSize + $spanSize >= self::$maxSpanBytes) {
self::$batchs[] = [
'thriftProcess' => $jaeger->processThrift,
Expand Down

0 comments on commit b5ed624

Please sign in to comment.