Skip to content

Fixing hadoop 3 Dockerfile#12284

Merged
abhishekagarwal87 merged 1 commit intoapache:masterfrom
cryptoe:hadoop3_profile
Feb 26, 2022
Merged

Fixing hadoop 3 Dockerfile#12284
abhishekagarwal87 merged 1 commit intoapache:masterfrom
cryptoe:hadoop3_profile

Conversation

@cryptoe
Copy link
Contributor

@cryptoe cryptoe commented Feb 25, 2022

As part of #11791 , the DockerFIle for hadoop 3 was referring to paths which are outside its build context. The fix here is to duplicate the template files across both hadoop/hadoop3 folders.
I am open to suggestion if there are other workarounds available.

Description

Fixed the bug ...

Renamed the class ...

Added a forbidden-apis entry ...


Key changed/added classes in this PR
  • dockerfile

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@abhishekagarwal87 abhishekagarwal87 merged commit a080fcd into apache:master Feb 26, 2022
@abhishekagarwal87 abhishekagarwal87 added this to the 0.23.0 milestone May 11, 2022
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.

2 participants