You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by hr...@apache.org on 2022/09/06 05:27:29 UTC
[pulsar-client-node] branch master updated: Update C++ client version and compatiblity table to v2.10.1 (#229)
This is an automated email from the ASF dual-hosted git repository.
hrsakai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-client-node.git
The following commit(s) were added to refs/heads/master by this push:
new db9ad97 Update C++ client version and compatiblity table to v2.10.1 (#229)
db9ad97 is described below
commit db9ad97091ef04e5687f32764f0037df2d23135b
Author: Yuri Mizushima <yu...@yahoo-corp.jp>
AuthorDate: Tue Sep 6 14:27:23 2022 +0900
Update C++ client version and compatiblity table to v2.10.1 (#229)
* chore: update pulsar-version to 2.10.1
* docs: update compatibility table
* chore: update default BUILD_IMAGE_VERSION to ubuntu-20.04
* test: modify expected exception message
---
README.md | 15 ++++++++-------
docker-tests.sh | 7 ++++---
pulsar-test-service-start.sh | 2 +-
pulsar-version.txt | 2 +-
run-unit-tests.sh | 2 +-
tests/consumer.test.js | 4 ++--
tests/producer.test.js | 4 ++--
7 files changed, 19 insertions(+), 17 deletions(-)
diff --git a/README.md b/README.md
index e29339d..88be059 100644
--- a/README.md
+++ b/README.md
@@ -40,13 +40,14 @@ Also, this library works only in Node.js 10.x or later because it uses the
Compatibility between each version of the Node.js client and the C++ client is as follows:
-| Node.js client | C++ client |
-|----------------|----------------|
-| 1.0.x | 2.3.0 or later |
-| 1.1.x | 2.4.0 or later |
-| 1.2.x | 2.5.0 or later |
-| 1.3.x | 2.7.0 or later |
-| 1.4.x - 1.6.x | 2.8.0 or later |
+| Node.js client | C++ client |
+|----------------|-----------------|
+| 1.0.x | 2.3.0 or later |
+| 1.1.x | 2.4.0 or later |
+| 1.2.x | 2.5.0 or later |
+| 1.3.x | 2.7.0 or later |
+| 1.4.x - 1.6.x | 2.8.0 or later |
+| 1.7.x | 2.10.1 or later |
If an incompatible version of the C++ client is installed, you may fail to build or run this library.
diff --git a/docker-tests.sh b/docker-tests.sh
index fab1042..3ba6b84 100755
--- a/docker-tests.sh
+++ b/docker-tests.sh
@@ -18,11 +18,11 @@
# under the License.
#
-ROOT_DIR=$(git rev-parse --show-toplevel)
+ROOT_DIR=${ROOT_DIR:-$(git rev-parse --show-toplevel)}
cd $ROOT_DIR
BUILD_IMAGE_NAME="${BUILD_IMAGE_NAME:-apachepulsar/pulsar-build}"
-BUILD_IMAGE_VERSION="${BUILD_IMAGE_VERSION:-ubuntu-16.04}"
+BUILD_IMAGE_VERSION="${BUILD_IMAGE_VERSION:-ubuntu-20.04}"
IMAGE="$BUILD_IMAGE_NAME:$BUILD_IMAGE_VERSION"
@@ -30,7 +30,8 @@ echo "---- Testing Pulsar node client using image $IMAGE"
docker pull $IMAGE
-DOCKER_CMD="docker run -i -v $ROOT_DIR:/pulsar-client-node $IMAGE"
+TARGET_DIR=/pulsar-client-node
+DOCKER_CMD="docker run -i -e ROOT_DIR=$TARGET_DIR -v $ROOT_DIR:$TARGET_DIR $IMAGE"
# Start Pulsar standalone instance
# and execute the tests
diff --git a/pulsar-test-service-start.sh b/pulsar-test-service-start.sh
index 44cdf9c..1054160 100755
--- a/pulsar-test-service-start.sh
+++ b/pulsar-test-service-start.sh
@@ -20,7 +20,7 @@
set -e
-ROOT_DIR=$(git rev-parse --show-toplevel)
+ROOT_DIR=${ROOT_DIR:-$(git rev-parse --show-toplevel)}
cd $ROOT_DIR
VERSION="${VERSION:-`cat ./pulsar-version.txt`}"
diff --git a/pulsar-version.txt b/pulsar-version.txt
index 834f262..8bbb6e4 100755
--- a/pulsar-version.txt
+++ b/pulsar-version.txt
@@ -1 +1 @@
-2.8.0
+2.10.1
diff --git a/run-unit-tests.sh b/run-unit-tests.sh
index 1033755..686ecf3 100755
--- a/run-unit-tests.sh
+++ b/run-unit-tests.sh
@@ -20,7 +20,7 @@
set -e
-ROOT_DIR=$(git rev-parse --show-toplevel)
+ROOT_DIR=${ROOT_DIR:-$(git rev-parse --show-toplevel)}
cd $ROOT_DIR
# install pulsar cpp client pkg
diff --git a/tests/consumer.test.js b/tests/consumer.test.js
index bcbcb61..b13fac2 100644
--- a/tests/consumer.test.js
+++ b/tests/consumer.test.js
@@ -102,7 +102,7 @@ const Pulsar = require('../index.js');
subscription: 'sub1',
ackTimeoutMs: 10000,
nAckRedeliverTimeoutMs: 60000,
- })).rejects.toThrow('Failed to create consumer: ConnectError');
+ })).rejects.toThrow('Failed to create consumer: BrokerMetadataError');
});
test('Not Exist Namespace', async () => {
@@ -111,7 +111,7 @@ const Pulsar = require('../index.js');
subscription: 'sub1',
ackTimeoutMs: 10000,
nAckRedeliverTimeoutMs: 60000,
- })).rejects.toThrow('Failed to create consumer: ConnectError');
+ })).rejects.toThrow('Failed to create consumer: BrokerMetadataError');
});
test('Not Positive NAckRedeliverTimeout', async () => {
diff --git a/tests/producer.test.js b/tests/producer.test.js
index a33318d..40712a5 100644
--- a/tests/producer.test.js
+++ b/tests/producer.test.js
@@ -55,7 +55,7 @@ const Pulsar = require('../index.js');
topic: 'persistent://no-tenant/namespace/topic',
sendTimeoutMs: 30000,
batchingEnabled: true,
- })).rejects.toThrow('Failed to create producer: ConnectError');
+ })).rejects.toThrow('Failed to create producer: BrokerMetadataError');
});
test('Not Exist Namespace', async () => {
@@ -63,7 +63,7 @@ const Pulsar = require('../index.js');
topic: 'persistent://public/no-namespace/topic',
sendTimeoutMs: 30000,
batchingEnabled: true,
- })).rejects.toThrow('Failed to create producer: ConnectError');
+ })).rejects.toThrow('Failed to create producer: BrokerMetadataError');
});
test('Automatic Producer Name', async () => {