[GitHub] ivankelly commented on issue #2865: Refactored JCloud Tiered Storage

2018-12-17 Thread GitBox
ivankelly commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-447984785 rerun java8 tests This is an automated message from the Apache Git Service. To

[GitHub] ivankelly commented on issue #2865: Refactored JCloud Tiered Storage

2018-12-17 Thread GitBox
ivankelly commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-447974459 rerun integration tests rerun java8 test This is an automated message from the

[GitHub] ivankelly commented on issue #2865: Refactored JCloud Tiered Storage

2018-12-12 Thread GitBox
ivankelly commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-446529648 Also, the old config values will still have to work. So you'll need some compatibility translation layer.

[GitHub] ivankelly commented on issue #2865: Refactored JCloud Tiered Storage

2018-12-12 Thread GitBox
ivankelly commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-446529215 I expect there will be non-jclouds offloaders too. For example, HDFS. [1] might be helpful as an example of doing weird stuff with the config.

[GitHub] ivankelly commented on issue #2865: Refactored JCloud Tiered Storage

2018-12-11 Thread GitBox
ivankelly commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-446342317 There will likely have to be some vendor specific things, as they don't have 100% compatible features. How about something like: ```

[GitHub] ivankelly commented on issue #2865: Refactored JCloud Tiered Storage

2018-12-11 Thread GitBox
ivankelly commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-446212610 > After making changes to the integration test property This is a BC break. A configuration which works with 2.1 or 2.2 will not work with this