Final review for PubSub by Google PubSub team

Should we have this reviewed more?

Yes, the doc is quite a mess actually, looking at the code is essential.

Could you please update the doc, the message.id should be message.ackId https://googlecloudplatform.github.io/gcloud-node/#/docs/v0.13.2/pubsub/subscription?method=ack

Thank you

@pierre-b

@pierre-b Hi thanks for your concerns. Is there any other specific issues you would like us to address?

@ryanseys

Hi @ryanseys , I noticed a 30sec average lag when I open a subscription and the first message arrives, then the others arrive directly. Is this an expected behavior ?

I also need to know how many messages are waiting in the subscription, to pop new workers or monitor the health of my system. Is it in the roadmap actually ?

Thank you

@pierre-b

Hi @pierre-b: Can you put these in separate issues so we can track them individually?

I will look at the code this week.

@jgeewax

Hi @jgeewax , forgot to tell you Ive opened separate issues.

Pubsub holds the project id, which prevents you from accessing resources in multiple projects.

https://github.com/GoogleCloudPlatform/gcloud-node/blob/master/lib/pubsub/index.js#L90

Is this expected?

Just to be clear, it seems to me that I could do:

var gcloud = require('gcloud');
var pubsub1 = gcloud.pubsub({projectId: 'project1'});
var pubsub2 = gcloud.pubsub({projectId: 'project2'});

right?

This would allow us to access resources across projects, no?

@jgeewax

@jgeewax I hope you're right. I meant you can not access multiple projects unless creating multiple pubsub object.

However, I came up with a bad scenario.
I have 2 projects; project1 and project2.
I have 1 topic; topic1 in project1.
I have 1 subscription; subscription2 in project2, which subscribes to the topic1.

I don't think I can access to the subscription2 easily. It seems to me that the only way is to call pubsub.getSubscriptions() and iterate like:

var pubsub2 = gcloud.pubsub({projectId: 'project2'});
pubsub2.getSubscriptions(
  func(err, subscriptions, nextQuery, apiResponse) {
    // I don't come up with how to iterate the result easily
  }
)

Since if I do the following, the subscription object will have a wrong projectId:

var pubsub1 = gcloud.pubsub({projectId: 'project1'});
var topic1 = pubsub1.topic('topic1');
var sub2 = topic1.subscription('subscription2'); // sub2 will have 'project1', which is wrong