Re: Janusgraph functionality issue
Dmitry Kovalev <dk.g...@...>
Hi Shiva,
unfortunately I can't help much on the CLA front, I am basically just your fellow contributor and don't manage CLAs. If you want to chase this specific item, probably the best place is janusgr...@... as suggested in the doc I posted the link to before.
Generally speaking, this project is supported by volunteers in their spare time, there doesn't seem to be anyone sponsored to work on it full time, so there may be any number of reasons for the delay in response to your contribution and CLA request. People could be just busy with their full-time work, or being affected by epidemic, or just having nice summer time off etc.
Similarly, when it comes to reviewing the actual contribution, many people (me included) get involved only occasionally, if they happen to be familiar with the relevant part of the codebase, or affected by the particular issue your contribution addresses. So it may take some time until a "more involved" team member can look at it and dig into the details.
Having been "in your shoes" before, I understand that delays may not exactly fill you with enthusiasm, but be assured that they don't mean your contribution is not appreciated - it will get looked at, and if it does add value, it will be merged eventually.
On Tue, 30 Jun 2020 at 09:37, <shivain...@...> wrote:
Hi Dmitry,--We have submitted a pull request and it got failed in CLA process.Our companies CLA Signatory raised a request to act as a CLA Manager from our company for this project.But the request got stuck at 'Awaiting Approval' since last few days. (Attached screenshot for reference)Could you please help us on this issue.
On Sunday, June 14, 2020 at 12:28:30 PM UTC+5:30, Shiva Krishnan wrote:Hi Dmitry,Thank you for the information.I will raise the PR as soon as possible.RegardsShivaOn Thursday, June 11, 2020 at 8:32:25 PM UTC+5:30, Dmitry Kovalev wrote:BTW, the doc suggests that you start with signing the CLA before forking and raising pull request. In case of individual contribution this is easy and fast, but for corporate route (which I am guessing you have to go through) it can be a long and unexciting part of contribution process.Likewise, getting your changes to be reviewed can also take time, depending on how big and complex your change is, and how many people are familiar enough with that part of code.So I suggest that you raise the PR as early in the process as possible, and the CLA bit can catch up (while it's still obviously required before your contribution can be finally accepted)On Thu, 11 Jun 2020 at 11:17, Dmitry Kovalev <d...@...> wrote:Hi Shiva,Thanks for taking time to dive into the issue and deciding to contribute.Please have a look at this page: https://github.com/JanusGraph/janusgraph/blob/master/CONTRIBUTING.md - it describes the steps to contribute, which are pretty standard.Thanks,DmitryOn Thu, 11 Jun 2020 at 10:40, <shi...@...> wrote:Hello Everyone,--I have found the root cause for this issue and also have the fix for the same.Please guide me if I can contribute for this project as I'm new to this community.ThanksShiva
On Monday, June 8, 2020 at 6:50:04 PM UTC+5:30, Shiva Krishnan wrote:Hi,I have noticed a corruption in index after reindexing vertex-centric indices with direction 'IN'.Issue : A self-link is getting added to every vertex after reindexing.To demonstrate this, I have taken a very small graph consists of two vertices (A & B) and one edge connecting the two.Edge Label -> linkEdge has one property key (Integer) -> assocKind//Creating the Vertex Centrix Indexgremlin > edgeLabel = mgmt.getEdgeLabel("link");gremlin > assocKind= mgmt.getPropertyKey("assocKind")gremlin > mgmt.buildEdgeIndex(edgeLabel, "myVertexCentricIndex", Direction.IN, assocKind);Please note i have given the direction IN for the index.//Creating the Edge from A to B: (a and b are vertices)a.addEdge("link",b,"assocKind",1)Output Before Reindexing :gremlin> g.V().has('name' , 'A').inE().hasLabel('link').has('assocKind',1)//no IN edges to vertex A Correctgremlin> g.V().has('name' , 'A').outE().hasLabel('link').has('assocKind',1)==>e[4e1f-b6g-1bit-1pqw][14488-link->80024] CorrectNow I ran the reindex command// 'index' is the vertex centric index which is created abovegremlin> m.updateIndex(index, SchemaAction.REINDEX).get()Output After Reindexing :gremlin> g.V().has('_objId','A').inE().hasLabel('link').has('assocKind',1)==>e[4e1f-b6g-1bit-b6g][14488-link->14488] Wronggremlin> g.V().has('_objId','A').bothE().hasLabel('link').has('assocKind',1)==>e[4e1f-b6g-1bit-1pqw][14488-link->80024]==>e[4e1f-b6g-1bit-b6g][14488-link->14488] Unexpected LinkThis issue happens in janusgraph 0.5.2 also.ThanksShiva
You received this message because you are subscribed to the Google Groups "JanusGraph developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jan...@....
To view this discussion on the web visit https://groups.google.com/d/msgid/janusgraph-dev/ea55b56a-dc8b-456e-b8c9-071de9b9f35ao%40googlegroups.com.
You received this message because you are subscribed to the Google Groups "JanusGraph developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to janusgr...@....
To view this discussion on the web visit https://groups.google.com/d/msgid/janusgraph-dev/4db2f58b-cd2d-406e-a6bb-bfb14e91b6edo%40googlegroups.com.
