Skip to content

Fixing typos in docker build scripts#11866

Merged
abhishekagarwal87 merged 1 commit intoapache:masterfrom
cryptoe:hadoop_3
Nov 2, 2021
Merged

Fixing typos in docker build scripts#11866
abhishekagarwal87 merged 1 commit intoapache:masterfrom
cryptoe:hadoop_3

Conversation

@cryptoe
Copy link
Contributor

@cryptoe cryptoe commented Nov 2, 2021

Fixes typos in the docker build script in case the Hadoop 3 profile is selected .

Description

I accidentally introduced a typo in the bash script as part of #11794. This bug does not get triggered in the default profile but will be triggered once we select the hadoop3 profile.

Fixed the bug ...

Renamed the class ...

Added a forbidden-apis entry ...


Key changed/added classes in this PR
  • build_run_cluster
  • copy_hadoop_resources
  • copy_resources_template

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.

@cryptoe
Copy link
Contributor Author

cryptoe commented Nov 2, 2021

Looks like the CICD failures are unrelated

@abhishekagarwal87 abhishekagarwal87 merged commit cf27366 into apache:master Nov 2, 2021
@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.

3 participants