Remove AWS dependency

[fnothaft]

See discussion in bigdatagenomics/adam#690.

We depend on the AWS SDK for the Serializable AWS credentials. I believe that this is dead code.

[heuermh]

re: bigdatagenomics/adam#690 (comment)

Taking a quick look at this, it looks generally useful and as such might still make sense in a utils module. Was the old S3 Parquet loader you refer to replaced by something else?

[fnothaft]

What is useful about this? The AWS SDK? Any downstream project can just depend on AWS independently.

[heuermh]

I'm seeing classes CredentialsProperties, S3ByteAccess, S3FileLocator, SerializableAWSCredentials. Even if no longer used in ADAM they seem generally useful.

[fnothaft]

Ah, gotcha. I'm not much of a fan of letting dead code that adds a dependency stay in the repository. If we need the code at a later date, we can always revert the commit that removed it.

[heuermh]

ok, then there's just the binary compatibility argument, which I'm not sure applies to these 0.x-versioned projects yet :)

[fnothaft]

Yeah, we're not concerned about that until 1.x.

[fnothaft]

Closed by 2b4c154.