[GitHub] StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB

2018-08-13 Thread GitBox
StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB URL: https://github.com/apache/flink/pull/6452#issuecomment-412566310 @sampathBhat There are two very different config parameter types: URIs across file systems (like where checkpoints go

[GitHub] StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB

2018-08-13 Thread GitBox
StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB URL: https://github.com/apache/flink/pull/6452#issuecomment-412565039 Given this implication on Yarn / Mesos, I am +1 to remove the absolute path requirement (my mistake to introduce that in

[GitHub] StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB

2018-08-13 Thread GitBox
StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB URL: https://github.com/apache/flink/pull/6452#issuecomment-412564568 My understanding is that relative paths may make sense in setups like YARN or Mesos, where the exact temp directories

[GitHub] StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB

2018-08-13 Thread GitBox
StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB URL: https://github.com/apache/flink/pull/6452#issuecomment-412559130 Do we want to restore the previous behavior? I mean, RocksDB paths must be local file paths. Having URIs with file

[GitHub] StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB

2018-07-30 Thread GitBox
StephanEwen commented on issue #6452: [FLINK-9739] Regression in supported filesystems for RocksDB URL: https://github.com/apache/flink/pull/6452#issuecomment-408945091 The character `:` is actually the standard UNIX path separator. Not supporting this would be kind of strange and break