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

Update tars client impl #821

Merged
merged 17 commits into from
Sep 12, 2023
Merged

Conversation

morebtcg
Copy link
Member

Update tars client impl

@sonarcloud
Copy link

sonarcloud bot commented Sep 12, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

static final int queueSize = 10 * 10000;
static final String libFileName = System.mapLibraryName("bcos_swig_java");

private class CallbackContent {

Check notice

Code scanning / CodeQL

Inner class could be static Note

CallbackContent should be made static, since the enclosing instance is not used.
() -> {
while (true) {
int seq = concurrentQueue.pop();
logger.debug("Receive queue message...", seq);

Check warning

Code scanning / CodeQL

Unused format argument Warning

This format call refers to 0 argument(s) but supplies 1 argument(s).
@@ -101,25 +152,27 @@
String signedTransactionData,
boolean withProof,
TransactionCallback callback) {
logger.debug("sendTransactionAsync...", node, withProof);

Check warning

Code scanning / CodeQL

Unused format argument Warning

This format call refers to 0 argument(s) but supplies 2 argument(s).
@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Patch has no changes to coverable lines.

❗ Current head 87b69c4 differs from pull request most recent head 59f630a. Consider uploading reports for the commit 59f630a to get more accurate results

📢 Thoughts on this report? Let us know!.

@kyonRay kyonRay merged commit c9012ea into FISCO-BCOS:release-3.5.0 Sep 12, 2023
8 of 9 checks passed
@morebtcg morebtcg deleted the release-3.5.0 branch September 12, 2023 03:28
kyonRay pushed a commit to kyonRay/java-sdk that referenced this pull request Oct 12, 2023
* Add TarsClient

* Update bulid.gradle, add tars-sdk

* Update fisco-bcos-tars-sdk package name

* Using google code style

* Merge origin

* Revert changes of xml

* ExtraData use string type

* Update tars client

* Add tars config and impl

* Update loadLibrary

* Use google java format

* Update library path find

* Update tars client

---------

Co-authored-by: More <More@MSI>
Co-authored-by: More <[email protected]>
kyonRay added a commit that referenced this pull request Oct 12, 2023
* <feat>(rpc): add call with sign interface of rpc. (#771)

* <feat>(rpc): add call with sign interface of rpc.

* <test>(integration): add v3.3.0 node test.

* <fix>(CI): fix integration test of extra data and input. (#796)

* <fix>(codec): fix int type not count bit size bug. (#800)

* upgrade openjdk version

* <fix>(test): remove unnecessary dependency com.google.guava. (#811)

* Add tars sdk impl (#818)

Add tars sdk impl (#818)

* <feat&fix>(transaction): fix call with sign result not handle revert message bug, add system features. (#819)

* Update tars client impl (#821)

* Add TarsClient

* Update bulid.gradle, add tars-sdk

* Update fisco-bcos-tars-sdk package name

* Using google code style

* Merge origin

* Revert changes of xml

* ExtraData use string type

* Update tars client

* Add tars config and impl

* Update loadLibrary

* Use google java format

* Update library path find

* Update tars client

---------

Co-authored-by: More <More@MSI>
Co-authored-by: More <[email protected]>

* <feat>(client): add getNodeListByType interface. (#822)

* Fix tars bug (#823)

* Add TarsClient

* Update bulid.gradle, add tars-sdk

* Update fisco-bcos-tars-sdk package name

* <fix>(client): add default nodeName to send request. (#825)

* Support jni inside jar (#826)

* Update sdk usage

* Support jni inside jar

---------

Co-authored-by: wenlinli <[email protected]>
Co-authored-by: MO NAN <[email protected]>
Co-authored-by: More <More@MSI>
Co-authored-by: More <[email protected]>
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.

2 participants