search

maheshwarirohit87 / typica

Automatically exported from code.google.com/p/typica
Apache License 2.0
0 stars 0 forks source link

Patch for InstanceType #34

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Improved the enum type to have more information about the instance type.
Applications need them to make intelligent decisions.

Original issue reported on code.google.com by kohsuke....@gmail.com on 30 Oct 2008 at 9:04

Attachments:

GoogleCodeExporter commented 9 years ago 
I'll just jot dowm some thoughts.
With enums like this, there was a strong temptation to leave it as a String type
since an enum requires a change to the library to support new instance types 
that
Amazon comes out with. Of course, they would probably change the API version at 
the
same time, so perhaps the point is moot.
With regard to this patch, I hesitate to put more non-api related information 
in the
code, but I also tend towards adding value on top of the APIs. My initial 
reaction is
that the computeUnits and cpuBits are helpful, but (while the costs haven't 
changed)
pricing might not fit as well.

Original comment by dkavan...@gmail.com on 20 Nov 2008 at 3:00

GoogleCodeExporter commented 9 years ago 
I've changed this to "WontFix" since I didn't get any input in almost a year on 
this one.

Original comment by dkavan...@gmail.com on 7 Oct 2009 at 1:14