removed logging of tokens #420
Conversation
Codecov Report
@@ Coverage Diff @@
## master #420 +/- ##
========================================
Coverage 86.97% 86.97%
========================================
Files 337 337
Lines 11078 11078
Branches 369 597 +228
========================================
Hits 9635 9635
Misses 1443 1443
Continue to review full report at Codecov.
|
| @@ -167,7 +167,6 @@ case class FeatureDistribution | |||
| "nulls" -> nulls.toString, | |||
| "distribution" -> distribution.mkString("[", ",", "]"), | |||
| "summaryInfo" -> summaryInfo.mkString("[", ",", "]"), | |||
| "cardinality" -> cardEstimate.map(_.toString).getOrElse(""), | |||
There was a problem hiding this comment.
@leahmcguire was there any value in what @TuanNguyen27 was doing in e0ea2ed#diff-528878f8bd336b485dd82f1fbef950f7R281
seq.map(_.size.toString)
which we could still do in toString
|
Thanks for the contribution! Unfortunately we can't verify the commit author(s): leahmcguire <l***@s***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, refresh the status of this Pull Request. |
Related issues
Refer to issue(s) addressed in this pull request from Issues page.
Describe the proposed solution
A clear and concise description of what the changes are.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context about the changes here.