Azure / azure-kusto-spark

Apache Spark Connector for Azure Kusto
Apache License 2.0
77 stars 34 forks source link

Handle spark's `BinaryType` #370

Closed fishfood34 closed 2 months ago

fishfood34 commented 6 months ago

Pull Request Description

Today BinaryType columns get written as a string by just using .toString on a byte array resulting in a value similar to [B@7fdcec67. This PR updates the handling of this column type to write a base64 encoded string instead.


Future Release Comment

[Add description of your change, to include in the next release] [Delete any or all irrelevant sections, e.g. if your change does not warrant a release comment at all]

Breaking Changes:

Features:

Fixes:

fishfood34 commented 6 months ago

@microsoft-github-policy-service agree company="GitHub"

ag-ramachandran commented 6 months ago

Hello @fishfood34 , Thanks for the contribution. Looks good to me overall. Couple of points

fishfood34 commented 6 months ago

@ag-ramachandran I believe I have made the requested changes. Please let me know if you see anything else I missed.