> @@ -345,7 +345,7 @@ protected UsageRecord(String id, @Nullable String
> description, @Nullable String
> @Nullable String releaseDate, @Nullable String
> zoneId, @Nullable String virtualMachineId, @Nullable String
> virtualMachineName,
> @Nullable String serviceOfferingId, @Nullable
> String templateId, @Nullable String ipAddress,
> boolean isSourceNAT, double rawUsageHours,
> @Nullable String usage, @Nullable String type, @Nullable UsageType usageType)
> {
> - this.id = checkNotNull(id, "id");
> + this.id = id;
Why this change? If `id` is nullable, should the method parameter not be
annotated? If it's **not** nullable, why omit the check?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/106/files#r5888068
