PHP thin client

[Pavel Petroshenko]

Hi Stepan,

I submitted a patch and updated JIRA with some comments. Can you please
give it a try?

Thanks,
P.


On Fri, Oct 19, 2018 at 9:07 AM Stepan Pilschikov <[hidden email]>
wrote:

> Alexey,
>
> Hi
>
> I'm trying to use PHP client for some times and discover one problem with
> DATA and TIMESTAMP data types
> I'ts appear when you trying to use DATA or TIMESTAMP as cache key
>
> All description in ticket:
> https://issues.apache.org/jira/browse/IGNITE-9951
>
> Can you help with that, please?
>
> чт, 11 окт. 2018 г. в 16:02, Igor Sapego <[hidden email]>:
>
> > Alexey,
> >
> > Since PHP thin client is now in master, can you please add it to our
> > "Thin clients features" wiki page, so we can track the feature parity
> > of our clients?
> >
> > [1] -
> > https://cwiki.apache.org/confluence/display/IGNITE/Thin+clients+features
> >
> > Best Regards,
> > Igor
> >
> >
> > On Wed, Oct 10, 2018 at 2:56 PM Vladimir Ozerov <[hidden email]>
> > wrote:
> >
> > > I am not sure there is an answer. This is just how we handled it
> before.
> > > IMO both approaches works fine. BTW, I already merged Vyacheslav's PR
> > > several hours ago.
> > >
> > > On Wed, Oct 10, 2018 at 1:24 PM Igor Sapego <[hidden email]>
> wrote:
> > >
> > >> Vyacheslav,
> > >>
> > >> In this case, we should do the same with all existing doxygen
> > >> files, not only this one (see files with .dxg extension)
> > >>
> > >> Vova, Pavel,
> > >>
> > >> Were there historically any reasons to exclude doxygen files
> > >> from check instead of adding a license header?
> > >>
> > >> Best Regards,
> > >> Igor
> > >>
> > >>
> > >> On Tue, Oct 9, 2018 at 11:49 PM Vyacheslav Daradur <
> [hidden email]
> > >
> > >> wrote:
> > >>
> > >>> Great work guys, thanks!
> > >>>
> > >>> I've noticed TC fails in the build plan "Licenses Headers" which Igor
> > >>> have fixed already by excluding Doxyfile from checking.
> > >>>
> > >>> I've filled a ticket [1] and I'd suggest fixing the test in another
> > >>> way by adding the license header to Doxyfile.
> > >>>
> > >>> Igor, please, have a look at prepared PR [2] (tests passed [3]).
> > >>>
> > >>> [1] https://issues.apache.org/jira/browse/IGNITE-9832
> > >>> [2] https://github.com/apache/ignite/pull/4938/files
> > >>> [3] https://ci.ignite.apache.org/viewLog.html?buildId=2046206
> > >>> On Tue, Oct 9, 2018 at 6:31 PM Igor Sapego <[hidden email]>
> wrote:
> > >>> >
> > >>> > As the seem to be everything OK with the code and tests and
> > >>> > no one had any comment for quite some time, I've merged the
> > >>> > PHP to Ignite's master.
> > >>> >
> > >>> > Great job, guys!
> > >>> >
> > >>> >
> > >>> > Best Regards,
> > >>> > Igor
> > >>> >
> > >>> >
> > >>> > On Wed, Sep 19, 2018 at 1:00 AM Prachi Garg <[hidden email]>
> > >>> wrote:
> > >>> >
> > >>> > > Hi Stephan, Alexey
> > >>> > >
> > >>> > > That's exactly what readme.io contains - installation
> > instructions,
> > >>> > > configuration, and examples for key-value, sql, etc. for thin
> > >>> clients. For
> > >>> > > example, see these documentation pages for Node.js (currently
> > hidden
> > >>> in the
> > >>> > > latest version of the doc) :
> > >>> > >
> > >>> > > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client
> > >>> > >
> > >>> > >
> > >>>
> >
> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-initialization-and-configuration
> > >>> > >
> > >>>
> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-key-value
> > >>> > > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-sql
> > >>> > >
> > >>>
> > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-binary-types
> > >>> > >
> > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-security
> > >>> > >
> > >>> > > This how Python and PHP thin clients will also be documented on
> > >>> readme.io
> > >>> > >
> > >>> > > -Prachi
> > >>> > >
> > >>> > >
> > >>> > >
> > >>> > >
> > >>> > > On Tue, Sep 18, 2018 at 3:26 AM, Stepan Pilshchikov <
> > >>> > > [hidden email]> wrote:
> > >>> > >
> > >>> > > > > You know, I'm confused with all this documentation stuff...
> > >>> > > > > For nodejs client all docs were moved from the repo to
> > readme.io
> > >>> but
> > >>> > > > the
> > >>> > > > > readme.md keeps the installation instructions (duplicated
> with
> > >>> > > > > readme.io). Probably, that's ok.
> > >>> > > > > Will add similar short readme.md to the PHP PR.
> > >>> > > >
> > >>> > > > Its good
> > >>> > > >
> > >>> > > > What i think (and how it partially now):
> > >>> > > > All user documentation should be on readme.io (how to install,
> > >>> use it,
> > >>> > > > configurate, description for examples)
> > >>> > > > All developers documentation (how to release, how to start
> > develop)
> > >>> > > and(!)
> > >>> > > > basic description should be in repository
> > >>> > > >
> > >>> > > >
> > >>> > > >
> > >>> > > >
> > >>> > > > --
> > >>> > > > Sent from:
> > http://apache-ignite-developers.2346864.n4.nabble.com/
> > >>> > > >
> > >>> > >
> > >>>
> > >>>
> > >>>
> > >>> --
> > >>> Best Regards, Vyacheslav D.
> > >>>
> > >>
> >
>

[Stepan Pilshchikov]

Thanks, finally fixes are merged

I have new question

I'm trying to put Enum object and get it back, but catch exception that
enum is null

code with output -
https://gist.github.com/pilshchikov/b89c6e25f29da88ac03caa42231a02bc

Also can't find Enum data type test in sources

I trying to use Enum in a right way? If answer is Yes, can you please check
this for yourself?
I think its a bug

вс, 21 окт. 2018 г. в 9:36, Pavel Petroshenko <[hidden email]>:

> Hi Stepan,
>
> I submitted a patch and updated JIRA with some comments. Can you please
> give it a try?
>
> Thanks,
> P.
>
>
> On Fri, Oct 19, 2018 at 9:07 AM Stepan Pilschikov <
> [hidden email]> wrote:
>
>> Alexey,
>>
>> Hi
>>
>> I'm trying to use PHP client for some times and discover one problem with
>> DATA and TIMESTAMP data types
>> I'ts appear when you trying to use DATA or TIMESTAMP as cache key
>>
>> All description in ticket:
>> https://issues.apache.org/jira/browse/IGNITE-9951
>>
>> Can you help with that, please?
>>
>> чт, 11 окт. 2018 г. в 16:02, Igor Sapego <[hidden email]>:
>>
>> > Alexey,
>> >
>> > Since PHP thin client is now in master, can you please add it to our
>> > "Thin clients features" wiki page, so we can track the feature parity
>> > of our clients?
>> >
>> > [1] -
>> >
>> https://cwiki.apache.org/confluence/display/IGNITE/Thin+clients+features
>> >
>> > Best Regards,
>> > Igor
>> >
>> >
>> > On Wed, Oct 10, 2018 at 2:56 PM Vladimir Ozerov <[hidden email]>
>> > wrote:
>> >
>> > > I am not sure there is an answer. This is just how we handled it
>> before.
>> > > IMO both approaches works fine. BTW, I already merged Vyacheslav's PR
>> > > several hours ago.
>> > >
>> > > On Wed, Oct 10, 2018 at 1:24 PM Igor Sapego <[hidden email]>
>> wrote:
>> > >
>> > >> Vyacheslav,
>> > >>
>> > >> In this case, we should do the same with all existing doxygen
>> > >> files, not only this one (see files with .dxg extension)
>> > >>
>> > >> Vova, Pavel,
>> > >>
>> > >> Were there historically any reasons to exclude doxygen files
>> > >> from check instead of adding a license header?
>> > >>
>> > >> Best Regards,
>> > >> Igor
>> > >>
>> > >>
>> > >> On Tue, Oct 9, 2018 at 11:49 PM Vyacheslav Daradur <
>> [hidden email]
>> > >
>> > >> wrote:
>> > >>
>> > >>> Great work guys, thanks!
>> > >>>
>> > >>> I've noticed TC fails in the build plan "Licenses Headers" which
>> Igor
>> > >>> have fixed already by excluding Doxyfile from checking.
>> > >>>
>> > >>> I've filled a ticket [1] and I'd suggest fixing the test in another
>> > >>> way by adding the license header to Doxyfile.
>> > >>>
>> > >>> Igor, please, have a look at prepared PR [2] (tests passed [3]).
>> > >>>
>> > >>> [1] https://issues.apache.org/jira/browse/IGNITE-9832
>> > >>> [2] https://github.com/apache/ignite/pull/4938/files
>> > >>> [3] https://ci.ignite.apache.org/viewLog.html?buildId=2046206
>> > >>> On Tue, Oct 9, 2018 at 6:31 PM Igor Sapego <[hidden email]>
>> wrote:
>> > >>> >
>> > >>> > As the seem to be everything OK with the code and tests and
>> > >>> > no one had any comment for quite some time, I've merged the
>> > >>> > PHP to Ignite's master.
>> > >>> >
>> > >>> > Great job, guys!
>> > >>> >
>> > >>> >
>> > >>> > Best Regards,
>> > >>> > Igor
>> > >>> >
>> > >>> >
>> > >>> > On Wed, Sep 19, 2018 at 1:00 AM Prachi Garg <[hidden email]>
>> > >>> wrote:
>> > >>> >
>> > >>> > > Hi Stephan, Alexey
>> > >>> > >
>> > >>> > > That's exactly what readme.io contains - installation
>> > instructions,
>> > >>> > > configuration, and examples for key-value, sql, etc. for thin
>> > >>> clients. For
>> > >>> > > example, see these documentation pages for Node.js (currently
>> > hidden
>> > >>> in the
>> > >>> > > latest version of the doc) :
>> > >>> > >
>> > >>> > > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client
>> > >>> > >
>> > >>> > >
>> > >>>
>> >
>> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-initialization-and-configuration
>> > >>> > >
>> > >>>
>> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-key-value
>> > >>> > > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-sql
>> > >>> > >
>> > >>>
>> >
>> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-binary-types
>> > >>> > >
>> > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-security
>> > >>> > >
>> > >>> > > This how Python and PHP thin clients will also be documented on
>> > >>> readme.io
>> > >>> > >
>> > >>> > > -Prachi
>> > >>> > >
>> > >>> > >
>> > >>> > >
>> > >>> > >
>> > >>> > > On Tue, Sep 18, 2018 at 3:26 AM, Stepan Pilshchikov <
>> > >>> > > [hidden email]> wrote:
>> > >>> > >
>> > >>> > > > > You know, I'm confused with all this documentation stuff...
>> > >>> > > > > For nodejs client all docs were moved from the repo to
>> > readme.io
>> > >>> but
>> > >>> > > > the
>> > >>> > > > > readme.md keeps the installation instructions (duplicated
>> with
>> > >>> > > > > readme.io). Probably, that's ok.
>> > >>> > > > > Will add similar short readme.md to the PHP PR.
>> > >>> > > >
>> > >>> > > > Its good
>> > >>> > > >
>> > >>> > > > What i think (and how it partially now):
>> > >>> > > > All user documentation should be on readme.io (how to
>> install,
>> > >>> use it,
>> > >>> > > > configurate, description for examples)
>> > >>> > > > All developers documentation (how to release, how to start
>> > develop)
>> > >>> > > and(!)
>> > >>> > > > basic description should be in repository
>> > >>> > > >
>> > >>> > > >
>> > >>> > > >
>> > >>> > > >
>> > >>> > > > --
>> > >>> > > > Sent from:
>> > http://apache-ignite-developers.2346864.n4.nabble.com/
>> > >>> > > >
>> > >>> > >
>> > >>>
>> > >>>
>> > >>>
>> > >>> --
>> > >>> Best Regards, Vyacheslav D.
>> > >>>
>> > >>
>> >
>>
>

[Stepan Pilshchikov]

And more,

If im use  ->setValueType(ObjectType::ENUM) during $cache->get(1);
it throwing exception that client can't convert BINARY_ENUM to ENUM

вт, 23 окт. 2018 г. в 15:57, Stepan Pilschikov <[hidden email]>:

> Thanks, finally fixes are merged
>
> I have new question
>
> I'm trying to put Enum object and get it back, but catch exception that
> enum is null
>
> code with output -
> https://gist.github.com/pilshchikov/b89c6e25f29da88ac03caa42231a02bc
>
> Also can't find Enum data type test in sources
>
> I trying to use Enum in a right way? If answer is Yes, can you please
> check this for yourself?
> I think its a bug
>
> вс, 21 окт. 2018 г. в 9:36, Pavel Petroshenko <[hidden email]>:
>
>> Hi Stepan,
>>
>> I submitted a patch and updated JIRA with some comments. Can you please
>> give it a try?
>>
>> Thanks,
>> P.
>>
>>
>> On Fri, Oct 19, 2018 at 9:07 AM Stepan Pilschikov <
>> [hidden email]> wrote:
>>
>>> Alexey,
>>>
>>> Hi
>>>
>>> I'm trying to use PHP client for some times and discover one problem with
>>> DATA and TIMESTAMP data types
>>> I'ts appear when you trying to use DATA or TIMESTAMP as cache key
>>>
>>> All description in ticket:
>>> https://issues.apache.org/jira/browse/IGNITE-9951
>>>
>>> Can you help with that, please?
>>>
>>> чт, 11 окт. 2018 г. в 16:02, Igor Sapego <[hidden email]>:
>>>
>>> > Alexey,
>>> >
>>> > Since PHP thin client is now in master, can you please add it to our
>>> > "Thin clients features" wiki page, so we can track the feature parity
>>> > of our clients?
>>> >
>>> > [1] -
>>> >
>>> https://cwiki.apache.org/confluence/display/IGNITE/Thin+clients+features
>>> >
>>> > Best Regards,
>>> > Igor
>>> >
>>> >
>>> > On Wed, Oct 10, 2018 at 2:56 PM Vladimir Ozerov <[hidden email]>
>>> > wrote:
>>> >
>>> > > I am not sure there is an answer. This is just how we handled it
>>> before.
>>> > > IMO both approaches works fine. BTW, I already merged Vyacheslav's PR
>>> > > several hours ago.
>>> > >
>>> > > On Wed, Oct 10, 2018 at 1:24 PM Igor Sapego <[hidden email]>
>>> wrote:
>>> > >
>>> > >> Vyacheslav,
>>> > >>
>>> > >> In this case, we should do the same with all existing doxygen
>>> > >> files, not only this one (see files with .dxg extension)
>>> > >>
>>> > >> Vova, Pavel,
>>> > >>
>>> > >> Were there historically any reasons to exclude doxygen files
>>> > >> from check instead of adding a license header?
>>> > >>
>>> > >> Best Regards,
>>> > >> Igor
>>> > >>
>>> > >>
>>> > >> On Tue, Oct 9, 2018 at 11:49 PM Vyacheslav Daradur <
>>> [hidden email]
>>> > >
>>> > >> wrote:
>>> > >>
>>> > >>> Great work guys, thanks!
>>> > >>>
>>> > >>> I've noticed TC fails in the build plan "Licenses Headers" which
>>> Igor
>>> > >>> have fixed already by excluding Doxyfile from checking.
>>> > >>>
>>> > >>> I've filled a ticket [1] and I'd suggest fixing the test in another
>>> > >>> way by adding the license header to Doxyfile.
>>> > >>>
>>> > >>> Igor, please, have a look at prepared PR [2] (tests passed [3]).
>>> > >>>
>>> > >>> [1] https://issues.apache.org/jira/browse/IGNITE-9832
>>> > >>> [2] https://github.com/apache/ignite/pull/4938/files
>>> > >>> [3] https://ci.ignite.apache.org/viewLog.html?buildId=2046206
>>> > >>> On Tue, Oct 9, 2018 at 6:31 PM Igor Sapego <[hidden email]>
>>> wrote:
>>> > >>> >
>>> > >>> > As the seem to be everything OK with the code and tests and
>>> > >>> > no one had any comment for quite some time, I've merged the
>>> > >>> > PHP to Ignite's master.
>>> > >>> >
>>> > >>> > Great job, guys!
>>> > >>> >
>>> > >>> >
>>> > >>> > Best Regards,
>>> > >>> > Igor
>>> > >>> >
>>> > >>> >
>>> > >>> > On Wed, Sep 19, 2018 at 1:00 AM Prachi Garg <[hidden email]>
>>> > >>> wrote:
>>> > >>> >
>>> > >>> > > Hi Stephan, Alexey
>>> > >>> > >
>>> > >>> > > That's exactly what readme.io contains - installation
>>> > instructions,
>>> > >>> > > configuration, and examples for key-value, sql, etc. for thin
>>> > >>> clients. For
>>> > >>> > > example, see these documentation pages for Node.js (currently
>>> > hidden
>>> > >>> in the
>>> > >>> > > latest version of the doc) :
>>> > >>> > >
>>> > >>> > > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client
>>> > >>> > >
>>> > >>> > >
>>> > >>>
>>> >
>>> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-initialization-and-configuration
>>> > >>> > >
>>> > >>>
>>> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-key-value
>>> > >>> > >
>>> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-sql
>>> > >>> > >
>>> > >>>
>>> >
>>> https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-binary-types
>>> > >>> > >
>>> > https://apacheignite.readme.io/v2.6/docs/nodejs-thin-client-security
>>> > >>> > >
>>> > >>> > > This how Python and PHP thin clients will also be documented on
>>> > >>> readme.io
>>> > >>> > >
>>> > >>> > > -Prachi
>>> > >>> > >
>>> > >>> > >
>>> > >>> > >
>>> > >>> > >
>>> > >>> > > On Tue, Sep 18, 2018 at 3:26 AM, Stepan Pilshchikov <
>>> > >>> > > [hidden email]> wrote:
>>> > >>> > >
>>> > >>> > > > > You know, I'm confused with all this documentation stuff...
>>> > >>> > > > > For nodejs client all docs were moved from the repo to
>>> > readme.io
>>> > >>> but
>>> > >>> > > > the
>>> > >>> > > > > readme.md keeps the installation instructions (duplicated
>>> with
>>> > >>> > > > > readme.io). Probably, that's ok.
>>> > >>> > > > > Will add similar short readme.md to the PHP PR.
>>> > >>> > > >
>>> > >>> > > > Its good
>>> > >>> > > >
>>> > >>> > > > What i think (and how it partially now):
>>> > >>> > > > All user documentation should be on readme.io (how to
>>> install,
>>> > >>> use it,
>>> > >>> > > > configurate, description for examples)
>>> > >>> > > > All developers documentation (how to release, how to start
>>> > develop)
>>> > >>> > > and(!)
>>> > >>> > > > basic description should be in repository
>>> > >>> > > >
>>> > >>> > > >
>>> > >>> > > >
>>> > >>> > > >
>>> > >>> > > > --
>>> > >>> > > > Sent from:
>>> > http://apache-ignite-developers.2346864.n4.nabble.com/
>>> > >>> > > >
>>> > >>> > >
>>> > >>>
>>> > >>>
>>> > >>>
>>> > >>> --
>>> > >>> Best Regards, Vyacheslav D.
>>> > >>>
>>> > >>
>>> >
>>>
>>