Issue with adding nested index dynamically

47 messages Options
Issue with adding nested index dynamically – We have to add indexes on cache dynamically on java pojo with nested objects. In the below example we do not have @QuerySqlField. In this case if...
Hi, thanks for starting this thread! SQL folks, could you please confirm that's neither known nor expected limitation? If that's a real issue ...
Hello! Please find/create a ticket about this issue and attach your patch to this ticket (in plain text or pull request form). Then it can ...
https://issues.apache.org/jira/browse/IGNITE-12261 I have created JIRA. Please let me know how I can assign to myself. My username on jira boa...
Hello! I have added you to Contributor role. Now you should be able to assign issue to yourself. Please read https://cwiki.apache.org/conf...
I have made changes and created pull request. Can you please check and let me know if there are any issues with the commit. Otherwise can you ple...
https://github.com/apache/ignite/pull/6949 Created pull request with suggestion. Can you please check and close -- Sent from: http://ap...
Hello! Can you add test case to this change? Otherwise, can you explain when it is useful? Can't zipcode just be annotated with @QuerySqlFi...
Sure, will add test case. My domain model is existing domain model and I cannot change or add @QuerySqlField to my domain model. I am trying ...
Added test case and created pull request. Please check both the code changes under pull request #6949 and test case pull request #6955 and let me...
Hello! Frankly speaking, I don't think this is a scenario that we want to support. If you are interested, you may like to start a discussion...
I have tested with adding QuerySqlFields as well, it is enabling to add dynamic field and indexes on nested objects but if I restart the client ...
ignite_bug_share.zip <http://apache-ignite-users.70518.x6.nabble.com/file/t2625/ignite_bug_share.zip> I have tested by adding QuerySql...
ignite_bug_share.zip <http://apache-ignite-users.70518.x6.nabble.com/file/t2625/ignite_bug_share.zip> I have tested by adding QuerySqlFie...
My apologies for multiple replies. Please consider latest reply -- Sent from: http://apache-ignite-users.70518.x6.nabble.com/
Hello! Currently we do not support altering tables which are created with indexedTypes/queryEntities. There are plans to implement it, but ...
Sorry for the push. This is a major blocker for us. We have coherence clients where they can add indexes dynamically and we want to move them to ...
Ivan, As a member of the Ignite SQL group, could you please check this thread and help Hemambara with a proper fix? - Denis On Thu,...
Hello Ivan, to be more specific, if I want add field "Users.userName" - current ignite version 2.7.6 is setting the field name as &...
Can you please provide any update in this... -- Sent from: http://apache-ignite-users.70518.x6.nabble.com/
Hello Ivan, can you please check and provide updates on this -- Sent from: http://apache-ignite-users.70518.x6.nabble.com/
Hi Hemambara, Could you bring a little bit more details about your problem? I would like to see you java classes and QueryEntity configuration...
Hello Denis/Ivan, can you please check this and if there are no issues can you merge the code. Sorry for the rush. We are getting ready for prod ...
I created a gist with my code [1] as it renders not so good inline. [1] https://gist.github.com/pavlukhin/362c0e40d4a010a8f7c0795368e53efb ч...
Please check Ivan's latest response - Denis On Thu, Oct 17, 2019 at 5:16 AM Hemambara <kotari4u@...> wrote: > Hello Denis/Iva...
ignite_bug_share.zip <http://apache-ignite-users.70518.x6.nabble.com/file/t2625/ignite_bug_share.zip> Thanks for the reply. Please fin...
Can you please provide an update -- Sent from: http://apache-ignite-users.70518.x6.nabble.com/
Hi Hemambara, I created an example [1] and checked with the proposed fix and without it. It seems the thing does not work in both cases. An...
I see there are couple of issues with the test 1) cache.query(new SqlFieldsQuery("alter table Person add column \"Address.street\&qu...
Hi Hemambara, Yep, I left some trash in gist [1] initially ("as address"), cleaned up. But does it really works as you need? In pr...
HI Ivan, thanks for the quick reply. Yes it perfectly works as needed with the fix. Person.Address.Street will not work because person.get...
Hi Hemambara, I rewrote my example in form of a junit test [1]. And I suppose that it should pass to make the feature useful. Could please che...
Result should not be same, could you please share your logs. Couple of issues here. 1) With out fix- I got null instead of "Address.stree...
Hi Hemambara, I tried adding 2nd node. I saw a following: 1. Without the fix 2nd is able to join cluster but failed to execute "select *&...
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...
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 &...
Sure. Have created one https://github.com/apache/ignite/pull/7016 Can you please check this. It has all the changes at one place. please let m...
Hello Ivan, please let me know if you get a chance to check the code and merge it.. -- Sent from: http://apache-ignite-users.70518.x6.nab...
Hi Hemambara, I checked some more scenarios with your fix. I a scenario: 1. Register QueryEntity with classes class Person { String name; Add...
I did not face any issue. Its working fine for me. Can you share your code and exception that you are getting I tried like below and it worked...
Hi Hemambara, I appologize but I will be able to share a problematic example only on next week. чт, 31 окт. 2019 г. в 19:49, Hemambara <...
Hi Hemambara, Check my example [1]. 1. Launch step1. 2. Uncomment code and enable Address.number field. 3. Launch step2. Actually the pro...
Okay, so the issue you are facing with is incorrect data type which is valid, so its not an issue then. Yes agreed that it requires more testi...
Hi Hemambara, You can a write an email to dev@... with reference to the issue and a description of the fix. It might be that someone will be ...
Hello, I tried to go to internals and override GridQueryProcessor but somehow I do not feel like clean solution. Just had another thought...whate...
Can anyone please check above comment and update. Thanks -- Sent from: http://apache-ignite-users.70518.x6.nabble.com/
Hello! I think you should move this discussion over to developers list. Regards, -- Ilya Kasnacheev вт, 26 нояб. 2019 г. в 15:01, He...