Skip to content

Commit bb9c532

Browse files
authored
fix(service-bus): removing namespaceType from NamespaceProperties (Azure#11995)
Addressing Azure#11956
1 parent b3720d8 commit bb9c532

File tree

6 files changed

+4
-9
lines changed

6 files changed

+4
-9
lines changed

sdk/servicebus/service-bus/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
- Added new "userId" property to `ServiceBusMessage` interface. [PR 11810](https://github.com/Azure/azure-sdk-for-js/pull/11810)
1616

1717
- `NamespaceProperties` interface property "messageSku" type changed from "string" to string literal type "Basic" | "Premium" | "Standard". [PR 11810](https://github.com/Azure/azure-sdk-for-js/pull/11810)
18+
- `NamespaceProperties` interface property "namespaceType" has been removed. [PR 11995](https://github.com/Azure/azure-sdk-for-js/pull/11995)
1819

1920
- Internal improvement - For the operations depending on `$management` link such as peek or lock renewals, the listeners for the "sender_error" and "receiver_error" events were added to the link for each new request made before the link is initialized which would have resulted in too many listeners and a warning such as `MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 sender_error listeners added to [Sender]. Use emittr.setMaxListeners() to increase limit`(same for `receiver_error`). This has been improved such that the listeners are reused.
2021
[PR 11738](https://github.com/Azure/azure-sdk-for-js/pull/11738)

sdk/servicebus/service-bus/review/service-bus.api.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,6 @@ export interface NamespaceProperties {
206206
messagingUnits: number | undefined;
207207
modifiedAt: Date;
208208
name: string;
209-
namespaceType: string;
210209
}
211210

212211
// @public

sdk/servicebus/service-bus/samples/javascript/advanced/administrationClient.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ async function main() {
4242
console.log("Number of messages in the queue = ", queueRuntimeProperties.totalMessageCount);
4343

4444
const namespaceInfo = await serviceBusAdministrationClient.getNamespaceProperties();
45-
console.log("Type of the namespace - ", namespaceInfo.namespaceType);
45+
console.log("Name of the namespace - ", namespaceInfo.name);
4646

4747
await serviceBusAdministrationClient.deleteQueue(queueName);
4848
const queueExists = await serviceBusAdministrationClient.queueExists(queueName);

sdk/servicebus/service-bus/samples/typescript/src/advanced/administrationClient.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ async function main() {
4343
console.log("Number of messages in the queue = ", queueRuntimeProperties.totalMessageCount);
4444

4545
const namespaceInfo = await serviceBusAdministrationClient.getNamespaceProperties();
46-
console.log("Type of the namespace - ", namespaceInfo.namespaceType);
46+
console.log("Name of the namespace - ", namespaceInfo.name);
4747

4848
await serviceBusAdministrationClient.deleteQueue(queueName);
4949
const queueExists = await serviceBusAdministrationClient.queueExists(queueName);

sdk/servicebus/service-bus/src/serializers/namespaceResourceSerializer.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,6 @@ export interface NamespaceProperties {
3333
* Name of the namespace.
3434
*/
3535
name: string;
36-
/**
37-
* Type of entities present in the namespace.
38-
*/
39-
namespaceType: string;
4036
/**
4137
* Number of messaging units allocated for namespace.
4238
* Valid only for Premium namespaces.
@@ -61,7 +57,6 @@ export function buildNamespace(rawNamespace: any): NamespaceProperties {
6157
messagingSku: messagingSku,
6258
modifiedAt: getDate(rawNamespace["ModifiedTime"], "modifiedAt"),
6359
name: getString(rawNamespace["Name"], "name"),
64-
namespaceType: getString(rawNamespace["NamespaceType"], "namespaceType"),
6560
messagingUnits:
6661
messagingSku === "Premium"
6762
? getInteger(rawNamespace["MessagingUnits"], "messagingUnits")

sdk/servicebus/service-bus/test/atomManagement.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ describe("Atom management - Namespace", function(): void {
6363
const namespaceProperties = await serviceBusAtomManagementClient.getNamespaceProperties();
6464
assert.deepEqualExcluding(
6565
namespaceProperties,
66-
{ messagingSku: "Standard", namespaceType: "Messaging", messagingUnits: undefined } as any,
66+
{ messagingSku: "Standard", messagingUnits: undefined } as any,
6767
["_response", "createdAt", "modifiedAt", "name"]
6868
);
6969
});

0 commit comments

Comments
 (0)