[
https://issues.apache.org/jira/browse/OLINGO-444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160294#comment-14160294
]
Ramesh Reddy commented on OLINGO-444:
-------------------------------------
Thanks for the explanation and commit.
1) As per the query options, I did not see $count on property, as it is always
one. $value is already counted for the interface. I will also consult V2 code
next time.
2) FIT module I do not think I saw any test code, anyway I can see if I submit
some tests based on it.
3) Context URL, yes new JIRA is better, I will open that. While working, I
always thought that why ContextURLHelper does not do the whole job of creating
the needed context-url, there isn't anything in the URL that processor
interface implementer needs to supply nor it is serializer specific. I was not
sure the reasoning behind the ODataSerializer.buildContextURLSelectList either.
(granted I have not looked into select or expand yet). In my code, I started
writing a totally independent ContextURL helper like
https://github.com/rareddy/teiid/blob/olingo/olingo/src/main/java/org/teiid/olingo/ContextURLHelper.java
If we think that is better route, I can certainly provide that as patch.
Obviously I will fix for completeness with testcases etc. Let me know.
> Addressing Property & $value is not supported
> ---------------------------------------------
>
> Key: OLINGO-444
> URL: https://issues.apache.org/jira/browse/OLINGO-444
> Project: Olingo
> Issue Type: New Feature
> Components: odata4-server
> Affects Versions: V4 4.0.0-beta-01
> Reporter: Ramesh Reddy
> Assignee: Michael Bolz
> Priority: Critical
> Fix For: (Java) V4 4.0.0-beta-02
>
>
> http://docs.oasis-open.org/odata/odata/v4.0/os/part2-url-conventions/odata-v4.0-os-part2-url-conventions.html#_Toc372793783
> http://docs.oasis-open.org/odata/odata/v4.0/os/part2-url-conventions/odata-v4.0-os-part2-url-conventions.html#_Toc372793784
> Both the above features are not supported
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)