Dmitry, If I understand correctly this ticket ( https://issues.apache.org/jira/browse/ZEPPELIN-545) assigned to Cos and has Fixed status.
The fix is ok. But removing of marshaller bean definition is better solution. So I can provide new PR or we can wait for Cos. On Mon, Jan 11, 2016 at 8:55 PM, Dmitriy Setrakyan <[email protected]> wrote: > Andrey, > > Any chance we could upgrade Zeppelin to work with Ignite 1.5? > > D. > > On Mon, Jan 11, 2016 at 7:44 AM, Andrey Gura <[email protected]> wrote: > > > 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 > > > -- Andrey Gura GridGain Systems, Inc. www.gridgain.com
