NAS and Job Feign Client #7
Conversation
| * @author Administrator | ||
| * | ||
| */ | ||
| @Lazy |
There was a problem hiding this comment.
Job is needed while performing most of ontap operations, so, is Lazy needed?
There was a problem hiding this comment.
Once this is in use, it will get initialise and next time it will be available so it will not go to initilize phase again so we can use lazy annotation, only SVM and cluster i did not require lazy bcz we need disaggregated and protocol info
| //File Operations | ||
|
|
||
| @RequestMapping(method = RequestMethod.GET, value="/{volume.uuid}/files/{path}") | ||
| OntapResponse<FileInfo> getFileResponse(URI uri, @RequestHeader("Authorization") String header, @PathVariable(name = "volume.uuid", required = true) String volumeUUID, |
There was a problem hiding this comment.
I remember seeing OnTapResponse, was this changed?
There was a problem hiding this comment.
yes changed to OntapResponse
| import org.springframework.web.bind.annotation.RequestMethod; | ||
|
|
||
|
|
||
| @FeignClient(name = "VolumeClient", url = "https://{clusterIP}/api/storage/volumes", configuration = FeignConfiguration.class) |
There was a problem hiding this comment.
I guess here we can use Lazy
| private Integer index = null; | ||
|
|
||
| public enum ProtocolsEnum { | ||
| any("any"), |
There was a problem hiding this comment.
Shall we leave them as is? Or shall we have only the ones that we would be supporting?
| } | ||
|
|
||
| @Override | ||
| public int hashCode() { |
There was a problem hiding this comment.
Could remove this too
| return o.toString().replace("\n", "\n "); | ||
| } | ||
|
|
||
| public static class ExportClient { |
There was a problem hiding this comment.
lets put this somewhere in beginning itself, instead at the EOF?
There was a problem hiding this comment.
Volume.java doesn't have a reference to Aggregate.java, please include it, if we are removing this
Also, can you please see if 'size' could be added?
|
|
||
| @Override | ||
| public int hashCode() { | ||
| return Objects.hash( id, name, rules, svm); |
There was a problem hiding this comment.
hashcode and equals should use same set of attributes to evaluate the usecase.
There was a problem hiding this comment.
sure, will include in next PR
| public enum ProtocolsEnum { | ||
| any("any"), | ||
|
|
||
| nfs("nfs"), |
There was a problem hiding this comment.
nfs is nfs3 only, do we really need nfs3 separately ?
| } | ||
|
|
||
| @Override | ||
| public int hashCode() { |
4 participants
Description
NAS and Job Feign Client with POJOs
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?