> @@ -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;
I think you mean https://github.com/jclouds/jclouds/pull/124? This contains
_two_ commits, is that the intention? Could you remove [this
commit](https://github.com/e-do/jclouds/commit/ce72bdda940d9b5039c26af5a3a6de0f1b36a916)
from the new PR, and also rename it to indicate what the purpose is, e.g.
"Mark id as nullable"? Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/106/files#r5967107
