Hi Hemambara, It really good that we have common understanding now. > I could not merge pull request as it is saying that only users with "Write > access" can merge pull requests
By merge I meant merging branches in your fork repository. It is really inconvenient to work with bunch of PRs simultaneously. It will be great if there is only one PR. пт, 25 окт. 2019 г. в 14:43, Hemambara <[email protected]>: > > Ohh yes you are right. Without the fix, it is failing to query cache. With > the other approach that I have attached earlier it failed to join node, > there I have CacheConfiguration in xml file for both server and client node. > Not sure if that does matters. Either way it is failing > > Also this is my bad, my apologies, that I forget to check in one change in > GridQueryProcessor.java where it is using QueryBinaryProperty constructor, > instead I am using QUeryUtils.buildBinaryProperty to build it. Now I have > checked in all the changes along with test cases. > > Can you please review pull requests: > https://github.com/apache/ignite/pull/7008 - Change in QueryUtils > https://github.com/apache/ignite/pull/7009 - Test case for QueryUtilsTest > https://github.com/apache/ignite/pull/7010 - Change in > GridQueryProcessor.java > https://github.com/apache/ignite/pull/7011 - Test case to make sure it is > adding nested field > > I could not merge pull request as it is saying that only users with "Write > access" can merge pull requests. Can you please merge these pull requests > and review. Please let me know if you see any issues. > > Appreciate your help on this. > > > > > > -- > Sent from: http://apache-ignite-users.70518.x6.nabble.com/ -- Best regards, Ivan Pavlukhin
