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.

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

Revise DynamicToArrow with existed engine schema of DynamicFragment #1501

Merged
merged 4 commits into from
May 6, 2022

Conversation

acezen
Copy link
Collaborator

@acezen acezen commented May 5, 2022

What do these changes do?

Related issue number

Fixes

@codecov-commenter
Copy link

codecov-commenter commented May 5, 2022

Codecov Report

Merging #1501 (40187f7) into main (779c93d) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1501      +/-   ##
==========================================
+ Coverage   73.59%   73.61%   +0.02%     
==========================================
  Files          88       88              
  Lines        9680     9680              
==========================================
+ Hits         7124     7126       +2     
+ Misses       2556     2554       -2     
Impacted Files Coverage Δ
python/graphscope/deploy/kubernetes/utils.py 70.47% <0.00%> (+0.95%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 779c93d...40187f7. Read the comment docs.

temp

typo

Fix typo

Move schema from client to engien

fix format

Update

Fix format

Fix

Update

Update

Update

Update

Update

Update

Update

Fix

Update

Format

Fix

Remove project test

Upgrade vineyard to 0.4.1 in Makefile (alibaba#1493)

Update

Test

Update

revert test

remove client lock

Update

Tests

Update

Update

pass compile

Replace collect property function with existed schema

Test

Fix

Fix

Update
@acezen acezen force-pushed the acezen/revise_dynamic_to_arrow branch from a6b47d9 to 466a551 Compare May 5, 2022 09:40
@acezen acezen marked this pull request as ready for review May 5, 2022 14:47
@acezen acezen merged commit 9357e13 into alibaba:main May 6, 2022
@acezen acezen deleted the acezen/revise_dynamic_to_arrow branch May 6, 2022 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants