Skip to content

Fix the issue with ADLSGen2 connector when writing a file#10230

Merged
snleee merged 1 commit intoapache:masterfrom
snleee:improve-adls-gen2
Feb 6, 2023
Merged

Fix the issue with ADLSGen2 connector when writing a file#10230
snleee merged 1 commit intoapache:masterfrom
snleee:improve-adls-gen2

Conversation

@snleee
Copy link
Contributor

@snleee snleee commented Feb 5, 2023

412 error has been introduced after the client version bump. This is because the behavior change from the ADLSGen2 client. For more context, please check:
Azure/azure-sdk-for-java#31248

412 error has been introduced after the client version bump.
This is because the behavior change from the ADLSGen2 client.
For more context, please check:
Azure/azure-sdk-for-java#31248
@codecov-commenter
Copy link

codecov-commenter commented Feb 5, 2023

Codecov Report

Merging #10230 (dc75316) into master (d63508d) will increase coverage by 1.90%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #10230      +/-   ##
============================================
+ Coverage     68.53%   70.43%   +1.90%     
+ Complexity     5126     5100      -26     
============================================
  Files          2000     2016      +16     
  Lines        108476   109154     +678     
  Branches      16484    16583      +99     
============================================
+ Hits          74341    76886    +2545     
+ Misses        28865    26892    -1973     
- Partials       5270     5376     +106     
Flag Coverage Δ
integration1 24.66% <ø> (-0.15%) ⬇️
integration2 24.50% <ø> (?)
unittests1 67.69% <ø> (-0.12%) ⬇️
unittests2 13.69% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ment/local/segment/index/column/NumValuesInfo.java 0.00% <0.00%> (-80.00%) ⬇️
...pinot/spi/exception/EarlyTerminationException.java 66.66% <0.00%> (-33.34%) ⬇️
...not/segment/spi/datasource/DataSourceMetadata.java 66.66% <0.00%> (-33.34%) ⬇️
...apache/pinot/minion/event/MinionEventObserver.java 25.00% <0.00%> (-25.00%) ⬇️
...a/org/apache/pinot/query/planner/PlannerUtils.java 75.00% <0.00%> (-25.00%) ⬇️
...ion/function/DistinctCountAggregationFunction.java 80.00% <0.00%> (-20.00%) ⬇️
...he/pinot/query/mailbox/InMemorySendingMailbox.java 57.14% <0.00%> (-15.59%) ⬇️
...g/apache/pinot/query/runtime/operator/OpChain.java 85.71% <0.00%> (-14.29%) ⬇️
...ation/function/DistinctSumAggregationFunction.java 85.71% <0.00%> (-14.29%) ⬇️
...al/segment/index/datasource/MutableDataSource.java 78.57% <0.00%> (-13.74%) ⬇️
... and 335 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@snleee snleee merged commit b50c58f into apache:master Feb 6, 2023
@snleee snleee deleted the improve-adls-gen2 branch February 6, 2023 01:39
@TanmayKedari
Copy link

In which Version was this issue fixed, started facing the issue again

@Jackie-Jiang
Copy link
Contributor

@TanmayKedari It was released in 0.12.1

@TanmayKedari
Copy link

azure-storage-file-datalake version is 12.8.0 in 0.12.1 version,
And Still facing the same while running in the 0.12.1 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants