igluctl: fix false failure message in push command

[chuwy]

When overriding an existing schema with 0.3.0 we're getting this error:

FAILURE: {
  "status" : 200,
  "message" : "Schema successfully updated",
  "location" : "/api/schemas/com.acme/event/jsonschema/1-0-0"
}

But upload actually works. 0.2.0 works without problems. It seems that problem is that Igluctl cannot deserialize above message into ServerMessage and ends up in this branch.

@oguzhanunlu, I believe this is same kind of problem we had with JSON Schema AST. Was that method also called asString?

[oguzhanunlu]

@chuwy , there were 2 methods and one of them was toString .

[chuwy]

Hey @oguzhanunlu, here's info you requested.

Sorry - I didn’t give you enough information about this one. Basically "schema registry" here is ours Scala Repo Server. You can assemble a fatjar and try to upload schemas for example from Iglu Central to it. Another way would be to use Snowplow Mini for it, but I think former option should be much faster. So, just launch a Scala Registry on your local machine and everything else should be easy.

[chuwy]

Hey @oguzhanunlu, just as heads-up, could you please check all iglu subprojects with Scala 2.12 for this json4s-extraction bug?

$ cd 0-common
$ grep -r extract\[ .

...or similar. We don't want to hit this in other places.

[oguzhanunlu]

@chuwy , trying to reproduce, I can't assemble scala repo server, I get java.lang.NoSuchMethodError: sbt.Keys$.compatibilityWarningOptions()Lsbt/SettingKey error after running sbt assembly. I use java 7 as pointed here . I wanted to download its jar directly but it is not specified yet. Can you assemble?

[chuwy]

@oguzhanunlu our compiled assets available at http://dl.bintray.com/snowplow/snowplow-generic/ (please fix the wiki page, also).

Problem most likely that you have SBT 1.0, while repo uses very old one. To avoid issues like this, please use Vagrantfile from project root. This is supposed to be default approach.

[oguzhanunlu]

@chuwy Your first assumption is true. Problem is asString. I also checked all Iglu subprojects for similar behavior and didn't encounter any.