## What is the purpose of the change

The S3 connectors are based on a self-contained shaded Hadoop. By design, they 
should only use config value from the Flink configuration.

However, because Hadoop loads implicitly configs from the classpath, existing 
"core-site.xml" files can interfere with the configuration in ways 
in-transparent for the user.

This PR ensure such configs are not loaded.

## Brief change log

  - The testing configs are renamed to "core-default-testing.xml"
  - The shaded adjusted config loading ignores "core-site.xml" files but loads 
"core-default-testing.xml" files

## Verifying this change

  - Includes a test by adding an improper "core-site.xml" to the test class 
path. If that file was loaded, the existing tests would fail.

## Does this pull request potentially affect one of the following parts:

  - Dependencies (does it add or upgrade a dependency): **no**
  - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: **no**
  - The serializers: **no**
  - The runtime per-record code paths (performance sensitive): **no**
  - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: **no**
  - The S3 file system connector: **yes**

## Documentation

  - Does this pull request introduce a new feature? **no**
  - If yes, how is the feature documented? **no**


[ Full content available at: https://github.com/apache/flink/pull/6728 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to