search

Azure / azure-kusto-spark

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

[Bugfix] Excluded included correct msal lib #360

Closed asaharn closed 9 months ago

asaharn commented 9 months ago

Pull Request Description

This fixes the classNotFound error, that comes with maven import of the connector ( note: works well with manual jar upload)

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:

github-actions[bot] commented 9 months ago

Test Results

0 tests  ±0   0 :white_check_mark: ±0   0s :stopwatch: ±0s 0 suites ±0   0 :zzz: ±0  0 files   ±0   0 :x: ±0 

Results for commit 6150a691. ± Comparison against base commit 8428f229.

:recycle: This comment has been updated with latest results.

asaharn commented 9 months ago

@ohadbitt most probably AzIdentity's msal version is getting replaced with the msal version that is imported via sdk

here is the log:

24/01/26 15:33:19 INFO LibraryResolutionManager: Library com/microsoft/azure/msal4j#1.13.10.jar is resolved with message: com/microsoft/azure/msal4j#1.13.9.jar has been evicted due to: [com/microsoft/azure/msal4j#1.13.10.jar]. 24/01/26 15:33:19 INFO LibraryResolutionManager: Library com/microsoft/azure/msal4j#1.13.10.jar is resolved with message: com/microsoft/azure/msal4j#1.4.0.jar has been evicted due to: [com/microsoft/azure/msal4j#1.13.10.jar].