Reverting back to netty 4.0 until there is a critical bug may be the best course. Spark 2.3 upgrades to netty 4.1 so perhaps that can be done in tp34/master.
Robert Dale On Tue, Mar 27, 2018 at 6:55 AM, Stephen Mallette <spmalle...@gmail.com> wrote: > hmm - i don't like how shifty this is all getting. I think I might bump > tp33 back to 4.0.x then everything will just work cleanly. > > btw, i'm also now aware of a second issue. the jackson version bump has > created an issue where byte[] is not serializing properly. it used to "just > work" but something must have changed in jackson to now require that it be > a registered type. i think i need to get that resolved prior to release - > this could mean a delay. > > On Tue, Mar 27, 2018 at 6:37 AM, Stephen Mallette <spmalle...@gmail.com> > wrote: > > > they would have a problem..... :/ > > > > i'm thinking this is an unlikely case? Spark based traversals are > > analytical and thus typically executed through the console rather than > via > > Gremlin Server. So perhaps not a big deal breaker? What do people think? > > > > That said, luckily the workaround is the same as the one for the console. > > Simply need to remove the netty 4.1.x jar from gremlin server's /lib and > > replace it with 4.0.x. I just ran the Gremlin Server integration tests > and > > they pass with that configuration. I could document that process in: > > > > https://github.com/apache/tinkerpop/blob/master/gremlin- > > server/conf/gremlin-server-spark.yaml > > > > > > > > On Mon, Mar 26, 2018 at 8:57 PM, Robert Dale <robd...@gmail.com> wrote: > > > >> What happens if someone wants to install Spark in Gremlin Server? > >> > >> Robert Dale > >> > >> On Mon, Mar 26, 2018 at 4:07 PM, Stephen Mallette <spmalle...@gmail.com > > > >> wrote: > >> > >> > Kuppitz pointed out that we were having trouble building docs on tp33 > >> and > >> > of course, it was spark related. I managed to sort out the problem and > >> we > >> > have our bump to netty 4.1.x to thank along with spark 2.2 being > pinned > >> on > >> > the 4.0.x line of netty. Basically spark is completely incompatible > >> there > >> > and we get errors as a result. I've tried all manner of things to > >> resolve > >> > it, but only one thing has actually worked. I had to pin the Gremlin > >> > Console to netty 4.0.x. Luckily, netty 4.0.x on gremlin-driver seems > >> > compatible with 4.1.x on the gremlin server so the console can still > >> talk > >> > to the server as needed. Here's the PR for consideration: > >> > > >> > https://github.com/apache/tinkerpop/pull/826 > >> > > >> > if anyone has a better way to resolve this, i'm open to going with it > - > >> in > >> > the mean time, if committers could review/vote, that would be good. > >> > > >> > > >> > > >> > On Sat, Mar 24, 2018 at 4:09 PM, Stephen Mallette < > spmalle...@gmail.com > >> > > >> > wrote: > >> > > >> > > Just a reminder that code freeze is in effect for the next week. > >> Please > >> > > post issues related to release of 3.2.8/3.3.2 on this thread as > >> needed. > >> > > > >> > > >> > > > > >