[GitHub] zeppelin issue #3001: [ZEPPELIN-3524] Replace the Amazon json libs with org....
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/3001 Sorry for the late reply. As you mentioned, Zeppelin already has two or more json dependencies at the same time, but I don't think it's good and we can fix it when we can do and have a chance. How about fixing it in this time? ---
[GitHub] zeppelin issue #3001: [ZEPPELIN-3524] Replace the Amazon json libs with org....
Github user arminwasicek commented on the issue: https://github.com/apache/zeppelin/pull/3001 The story is that Amazon deprecated their use of json.org and moved to jackson. The Zeppelin code in question uses on json.org through the import of the Amazon lib. The change I'm proposing is to explicitly include json.org, so everything else can stay the same. Using gson (or jackson via amazon for that matter) would make sense, but this entails changing more code. ---
[GitHub] zeppelin issue #3001: [ZEPPELIN-3524] Replace the Amazon json libs with org....
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/3001 @arminwasicek ping ---
[GitHub] zeppelin issue #3001: [ZEPPELIN-3524] Replace the Amazon json libs with org....
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/3001 BTW, this module already has `gson` as a dependency. Do you have any reason adding it? ---