Skip to content

Conversation

@ayushtkn
Copy link
Member

What changes were proposed in this pull request?

Use Local FS to qualify a path if LOCAL is used

Why are the changes needed?

To push correct qualified path to Ranger when using LOCAL

Does this PR introduce any user-facing change?

No

Is the change a dependency upgrade?

No

How was this patch tested?

Copy link
Member

@deniskuzZ deniskuzZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1, pending tests

@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

8 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@ayushtkn ayushtkn merged commit cdfada5 into apache:master Dec 21, 2023
tarak271 pushed a commit to tarak271/hive-1 that referenced this pull request Dec 22, 2023
…CTORY ops. (apache#4957). (Ayush Saxena, reviewed by Denys Kuzmenko)
tarak271 pushed a commit to tarak271/hive-1 that referenced this pull request Dec 22, 2023
…CTORY ops. (apache#4957). (Ayush Saxena, reviewed by Denys Kuzmenko)
deniskuzZ pushed a commit to deniskuzZ/hive that referenced this pull request Jan 2, 2024
…CTORY ops. (apache#4957). (Ayush Saxena, reviewed by Denys Kuzmenko)
dengzhhu653 pushed a commit to dengzhhu653/hive that referenced this pull request Mar 7, 2024
…CTORY ops. (apache#4957). (Ayush Saxena, reviewed by Denys Kuzmenko)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants