Kimahriman / hdfs-native

Apache License 2.0
33 stars 12 forks source link

`hdfs_native::Client::new_with_config` cannot be used because the `Configuration` type is not public. #65

Closed Xuanwo closed 10 months ago

Xuanwo commented 10 months ago

image

Do you want to expose Configuration directly? I'm willing to submit a PR for this.

zuston commented 10 months ago

Oh .yes. Feel free to fix this, I hope this could be fixed ASSP.

Kimahriman commented 10 months ago

Ah should have caught that, I assumed rust would have complained internally if that was the case. Lets just have that method take a HashMap instead? No reason to really expose the config right now I don't think

Xuanwo commented 10 months ago

Ah should have caught that, I assumed rust would have complained internally if that was the case. Lets just have that method take a HashMap instead? No reason to really expose the config right now I don't think

Looks nice to me. I will start a PR for this tomorrow.

Kimahriman commented 10 months ago

Resolved by https://github.com/Kimahriman/hdfs-native/pull/66