Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Janusupgrade to 1.0.0 with a smart marker logic to reduce Cassandra Calls #3876

Merged
merged 23 commits into from
Dec 11, 2024
Merged

Janusupgrade to 1.0.0 with a smart marker logic to reduce Cassandra Calls #3876

merged 23 commits into from
Dec 11, 2024

Conversation

aarshi0301
Copy link

Change description

Description here

  • Upgrade janusgraph to 1.0.0
  • Add a marker logic to lookup Cassandra only when property exist
  • Marker logic does not include IN ward relations to vertex
  • Uses Janus graph prefetch with properties

Type of change

  • Bug fix (fixes an issue)
  • New feature (adds functionality)

Related issues

Fix #1

Checklists

Development

  • Lint rules pass locally
  • Application changes have been tested thoroughly
  • Automated tests covering modified code pass

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
  • "Ready for review" label attached and reviewers assigned
  • Changes have been reviewed by at least one other contributor
  • Pull request linked to task tracker where applicable

sriram-atlan and others added 23 commits November 22, 2024 23:56
@sriram-atlan
multi query with janusgraph 1.0.0
0af18e1
@sriram-atlan
add attributes part of multi query
4e39d43
@sriram-atlan
include janusoptimisation-multiQuery-1.0.0 in ecr build push
9c5a643
@sriram-atlan
include janusoptimisation-1.0.0 in ecr build push
01e5b3b
@sriram-atlan
include janusupgrade in ecr build push
ee86e6e
@sriram-atlan
bulk fetch with auto batching
e042b4f
@sriram-atlan
use properties method with janusgraph than gremlin
9ee5645
@sriram-atlan
use edge fetch always
c8578e4
@sriram-atlan
log edge and edge properties
a47bdee
@sriram-atlan
lookup only when marker exists or when attribute relationship is inwards
4dd2743
@sriram-atlan
include all policy attributes in prefetch exclude
76fe1c8
@sriram-atlan
revert using prefetch in mapSystemAttributes and toAtlasToObjectId
358a7bc
@sriram-atlan
remove logs. update comments
5c6ad1b
@aarshi0301
fix business attribute
b95c0e6
@aarshi0301
add check for business attributes
4cb985a
@aarshi0301
code cleanup
8eef377
@aarshi0301
Merge branch 'janusupgrade' into dev/janusupgrade
18184d7
@aarshi0301
Merge branch 'beta' into dev/janusupgrade
9e77b09
@aarshi0301
fix duplicate
7f7f1b2
@aarshi0301
remove enable cache
4893830
@aarshi0301
remove test tools
3e06228
@aarshi0301
renove cache
d8c00b0
@aarshi0301
remove scope
7630c8c
@aarshi0301 aarshi0301 self-assigned this Dec 11, 2024
@aarshi0301 aarshi0301 merged commit 21e72d1 into beta Dec 11, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@aarshi0301 aarshi0301

Labels
beta
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aarshi0301 @sriram-atlan