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

style(store): translate Chinese comment to English #2614

Closed
wants to merge 11 commits into from

Conversation

Pengzna
Copy link
Contributor

@Pengzna Pengzna commented Aug 4, 2024

as title.

@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. store Store module labels Aug 4, 2024
Copy link

codecov bot commented Aug 4, 2024

Codecov Report

Attention: Patch coverage is 0% with 13 lines in your changes missing coverage. Please review.

Project coverage is 39.95%. Comparing base (91f5b33) to head (9d3a45c).

Files Patch % Lines
.../hugegraph/store/business/BusinessHandlerImpl.java 0.00% 3 Missing ⚠️
...org/apache/hugegraph/store/raft/RaftOperation.java 0.00% 3 Missing ⚠️
.../apache/hugegraph/store/meta/PartitionManager.java 0.00% 2 Missing ⚠️
...in/java/org/apache/hugegraph/store/meta/Store.java 0.00% 2 Missing ⚠️
...java/org/apache/hugegraph/store/HgStoreEngine.java 0.00% 1 Missing ⚠️
...va/org/apache/hugegraph/store/PartitionEngine.java 0.00% 1 Missing ⚠️
...ache/hugegraph/store/pd/FakePdServiceProvider.java 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (91f5b33) and HEAD (9d3a45c). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (91f5b33) HEAD (9d3a45c)
6 4
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2614      +/-   ##
============================================
- Coverage     47.12%   39.95%   -7.18%     
+ Complexity      583      532      -51     
============================================
  Files           718      729      +11     
  Lines         58441    59508    +1067     
  Branches       7492     7659     +167     
============================================
- Hits          27543    23775    -3768     
- Misses        28127    33221    +5094     
+ Partials       2771     2512     -259     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

*/
@Slf4j
public class HgThread2DB {

/*正在进行和在排队的任务的总数*/
/* The total number of tasks in line and queuing*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/* The total number of tasks in line and queuing*/
/* The total number of tasks in line and queuing */

Comment on lines +46 to +47
* This version isstore Internal use.Write data to the partition,
* Partitionid andkey.keycode must be consistent with the partition information of PD storage.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* This version isstore Internal use.Write data to the partition,
* Partitionid andkey.keycode must be consistent with the partition information of PD storage.
* This version is store internal use. Write data to the partition,
* partitionid and key.keycode must be consistent with the partition information of PD storage.

@@ -48,7 +48,7 @@ public HgStoreNodePartitionerImpl(PDClient pdClient, HgStoreNodeManager nodeMana
}

/**
* 查询分区信息,结果通过HgNodePartitionerBuilder返回
* Query Partition Information, the result is returned through HGNODEPARTITITITITINERBUILDER
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Query Partition Information, the result is returned through HGNODEPARTITITITITINERBUILDER
* Query Partition Information, the result is returned through HgNodePartitionerBuilder

@@ -141,7 +141,7 @@ public int partition(HgNodePartitionerBuilder builder, String graphName,
}

/**
* 查询hgstore信息
* Inquirehgstore information
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Inquirehgstore information
* Inquire hgstore information

@@ -692,7 +692,7 @@ public boolean doAction(String table, HgOwnerKey startKey, Integer code,
}

private List<NodeTkv> toNodeTkvList(Builder scanReqBuilder) {
// TODO 使用builder获取owner
// Todo use builder to get Owner
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// Todo use builder to get Owner
// TODO: use builder to get owner

@@ -170,7 +173,7 @@ public void add(Supplier<HgKvIterator<HgKvEntry>> supplier) {
}

/**
* 迭代器是否有效,如果没有数据,等待数据到达
* WHETHER the ITERATOR is value, if no data, wait for the data to arrive
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* WHETHER the ITERATOR is value, if no data, wait for the data to arrive
* whether the iterator is valid, if no data, wait for the data to arrive

@@ -221,18 +226,18 @@ private void moveNext() {
}

/**
* 对多个Scanner返回结果进行归并排序
* Multiplescanner return results for merger and sorting
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Multiplescanner return results for merger and sorting
* Multiple scanner return results for merger and sorting

@@ -153,7 +154,7 @@ public synchronized boolean init(final HgStoreEngineOptions opts) {
}

/**
* 创建raft rpc server,用于store之间通讯
* Createraft RPC Server, used for communication between store
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Createraft RPC Server, used for communication between store
* Create raft RPC Server, used for communication between store

partitionManager.setStore(store);
partitionManager.loadPartition();
restoreLocalPartitionEngine();
}
}

/**
* 恢复本地的PartitionEngine,恢复PD返回的分区信息
* 1、需要检查本次保存的分区,删除作废的分区
* RESTORE LOCALPARTITITITINGINE, restore the partition information returned by PD
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* RESTORE LOCALPARTITITITINGINE, restore the partition information returned by PD
* restore local PartitionEngine, restore the partition information returned by PD

*
* @param storeId 变更的store id
* @param storeId changedstore id
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* @param storeId changedstore id
* @param storeId changed store id

@VGalaxies
Copy link
Contributor

Considering the large number of files, can be split into multiple PRs...

Using a more effective translator and matching method...

@VGalaxies
Copy link
Contributor

close by #2623 and #2624

@VGalaxies VGalaxies closed this Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XL This PR changes 500-999 lines, ignoring generated files. store Store module
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants