René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Is MergeFrom used by accident (instead of CopyFrom) in bigtable.Table.create?","articleBody":"See [source][1]. As an example of why `MergeFrom` instead of `CopyFrom` can be a bug:\r\n\r\n```python\r\n\u003e\u003e\u003e from google.protobuf.timestamp_pb2 import Timestamp\r\n\u003e\u003e\u003e from google.cloud.bigtable._generated.bigtable_instance_admin_pb2 import (\r\n...     CreateInstanceMetadata)\r\n\u003e\u003e\u003e\r\n\u003e\u003e\u003e metadata1 = CreateInstanceMetadata(request_time=Timestamp(nanos=100))\r\n\u003e\u003e\u003e metadata2 = CreateInstanceMetadata(request_time=Timestamp(nanos=100))\r\n\u003e\u003e\u003e t = Timestamp(seconds=1000)\r\n\u003e\u003e\u003e metadata1\r\nrequest_time {\r\n  nanos: 100\r\n}\r\n\r\n\u003e\u003e\u003e metadata2\r\nrequest_time {\r\n  nanos: 100\r\n}\r\n\r\n\u003e\u003e\u003e t\r\nseconds: 1000\r\n\r\n\u003e\u003e\u003e\r\n\u003e\u003e\u003e metadata1.request_time.MergeFrom(t)\r\n\u003e\u003e\u003e metadata2.request_time.CopyFrom(t)\r\n\u003e\u003e\u003e\r\n\u003e\u003e\u003e metadata1\r\nrequest_time {\r\n  seconds: 1000\r\n  nanos: 100\r\n}\r\n\r\n\u003e\u003e\u003e metadata2\r\nrequest_time {\r\n  seconds: 1000\r\n}\r\n```\r\n\r\n[1]: https://github.com/GoogleCloudPlatform/google-cloud-python/blob/2d81f88454724f0a672b453c994413c3cf1110c7/bigtable/google/cloud/bigtable/table.py#L184","author":{"url":"https://github.com/dhermes","@type":"Person","name":"dhermes"},"datePublished":"2017-06-30T17:05:43.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":3},"url":"https://github.com/3571/google-cloud-python/issues/3571"}