NFData instances?

[gregwebs]

I wanted to put a retry around sending a request to AWS. Unfortunately there are no NFData instances, so it is difficult for me to confidently trap an exception.

Is it possible to automatically generate NFData instances for everything?

[brendanhay]

Sure is. Been meaning to add them for a while, so I'll do it sometime over the weekend.

[brendanhay]

Unfortunately due to the use of conduit for some request/response types, NFData instances cannot be provided. I've attempted to generate them wherever else possible.

[brendanhay]

Let me know if this will cover your use case @gregwebs, and I'll look at cutting a release pending some other minor changes + CI.

[gregwebs]

Seems like a great improvement. However, I still have many responses without instances. some of which seem like they shouldn't have anything to do with conduits.

• SetDesiredCapacityResponse
• TerminateInstancesResponse
• DescribeSpotPriceHistoryResponse
• DetachInstancesResponse

[gregwebs]

Note that I cannot generate instances myself because smart constructors and exported instead of the data constructors.

[brendanhay]

Thanks for catching that, an error on my part. d75a122 adds the expected response instances.

Does your latter comment imply you'd rather see fully exported types?

[gregwebs]

That's better!

This satisfies my only current use case for fully exported types. In general it can be nice to expose them at least from an Internal module so that a user can create instances.

[brendanhay]

I ask because it's not that I haven't exported them out of some misguided principle, it just requires some work to ensure that the .Internal style modules are generated - or that there isn't an additionally overwhelming amount of exports/imports (also Haddock) happening. I'll look into doing it as a somewhat low priority.

[brendanhay]

The NFData instances are available in 1.4.1 on Hackage.