Cos, the problem is that default config (incubator-zeppelin/ignite/src/main/resources/default-ignite-jdbc.xml) contains `marshaller` bean definition of OptimizedMarshaller type. IgniteSqlInterpreter class uses this config. Just remove this bean definition and default marshaller will be used.
On Mon, Jan 11, 2016 at 11:57 AM, Yakov Zhdanov <[email protected]> wrote: > Andrey Gura, can you please clarify? > > --Yakov > > 2016-01-06 22:06 GMT+03:00 Konstantin Boudnik <[email protected]>: > > > On Tue, Jan 05, 2016 at 10:40PM, Dmitriy Setrakyan wrote: > > > On Tue, Jan 5, 2016 at 8:12 PM, Konstantin Boudnik <[email protected]> > > wrote: > > > > > > > Test is pretty simple and doesn't do anything crazy at least in my > > > > uneducated > > > > opinion. > > > > > > > > To Yakov's point about using default binary marshaller: it is unclear > > why > > > > the > > > > client node won't use the default then. The test was working with 1.4 > > and > > > > then > > > > stopped. It hasn't been touched for a while, which tells me that > > something > > > > got > > > > changed in the underlying library. > > > > > > > > > > Cos, I think you are right. Most likely the client code in Zeppelin > sets > > up > > > the marshaller. We need to update it to work with 1.5. We should also > > > should allow to specify a default cache per notebook, so users could > > > override the default cache setting in the interpreter. > > > > Looking at the IgniteInterpreter code, again just briefly, there's no way > > to > > set the marshaller from the test. But I let ppl more familiar with the > > matter > > to chime in. For now the test is fixed, although clumsily. > > > > > > Thanks for looking into this! > > > > cos > > > > > > > > On Tue, Jan 05, 2016 at 01:46PM, Alexey Goncharuk wrote: > > > > > Server/client mode does not (and should not) change the marshaller > > > > > settings, to my knowledge there are no such places in the code that > > might > > > > > change this behavior. > > > > > > > > > > There might be something in the Zep integration test, but I need > some > > > > time > > > > > to install Zep and take a look at the code. > > > > > > > -- Andrey Gura GridGain Systems, Inc. www.gridgain.com
