Lukasz Stefaniak comments

Results 12 comments of


                                            Lukasz Stefaniak

replace spray with akka http may be better

Done here: https://github.com/sclasen/akka-zk-cluster-seed/pull/22

replace spray with akka http may be better

Hmmm, but we don't use any akka-http extensions for spray-json, it is still used directly so that change only adds more dependencies

TrustManager does not verify certificate or hostname

After release with akka-http it will work correctly using https://github.com/typesafehub/ssl-config/blob/master/ssl-config-core/src/main/scala/com/typesafe/sslconfig/ssl/DefaultHostnameVerifier.scala

similar projects

Hi @ouven @MasseGuillaume To use akka-viz you don't need to recompile anything, we use runtime weaving, all you need is just add akka-viz jar dependency and make sure you run...

Error : spray.json.SerializationException: Map key must be formatted as JsString, not '["A","b"]'

That would work: ``` object Foo { import spray.json._ case class CampKey(value: String) extends AnyVal case class DataPointKey(value: String) extends AnyVal case class DataPointValue(value: String) extends AnyVal type DataMap =...

Error : spray.json.SerializationException: Map key must be formatted as JsString, not '["A","b"]'

And you should probably change reads, to get rid of asInstanceOf, and fail with `DeserializationException` using `deserializationError` if value is not JsString.

Use google/cloud-sdk emulator to add integration tests for Google Cloud Storage

I don't think there is Cloud Storage emulator available. ``` Available groups for gcloud beta emulators: bigtable *(BETA)* Manage your local Bigtable emulator. datastore *(BETA)* Manage your local datastore emulator....

Go SDK string returning issue

The same problem happens when using `/queries/%d/run/sql` - it tries to decode JSON but inside there is just a string. `/content_thumbnail/look/X/png` - instead of returning `[]byte` it returns empty string

Add Location/Spans in to the AST/parse layer

I used this PR in our forked https://github.com/getsynq/sqlparser-rs and a tool built on top of that, and it works amazingly well. Adding `empty_span()` all over the place is painful initially,...

Add Location/Spans in to the AST/parse layer

After working on column-level lineage using sqlparser-rs even more, I think we must add location to every instance of `Ident` and maybe also `ObjectName` as otherwise it is super annoying...