In the past we built HBase against Hadoop 2.x, in which the Configuration object did not yet support the syntax for using environment variables in config files: ${env.VARNAME}
To work around this we added sed calls to do the replacement in the config files during container startup.
Since we now build HBase with Hadoop 3.x this functionality is now supported by HBase and we don't need these calls to sed anymore, as the placeholders will be replaced by HBase at runtime.
This PR removes the now unneeded calls.
Definition of Done Checklist
Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
Please make sure all these things are done and tick the boxes
# Author
- [x] Helm chart can be installed and deployed operator works
- [x] Integration tests passed (for non trivial changes)
Description
In the past we built HBase against Hadoop 2.x, in which the Configuration object did not yet support the syntax for using environment variables in config files:
${env.VARNAME}To work around this we added sed calls to do the replacement in the config files during container startup.
Since we now build HBase with Hadoop 3.x this functionality is now supported by HBase and we don't need these calls to sed anymore, as the placeholders will be replaced by HBase at runtime.
This PR removes the now unneeded calls.
Definition of Done Checklist