Commit d9d3390
authored
allow ignore nested fieldpaths (#260)
When implementing the update code paths for OpenSearchService, I found
out I needed to ignore the AutoTuneOptions.RollbackOnDisable nested
field within the UpdateDomainConfigInput shape. When I went to put the
following in the `generator.yaml` file:
```yaml
ignore:
field_paths:
- UpdateDomainConfigInput.AutoTuneOptions.RollbackOnDisable
```
What ended up happening is that the *entire* AutoTuneOptions field was
ignored instead of just the RollbackOnDisable field.
This patch fixes this situation by modifying the
`pkg/model.Model:ApplyIgnoreShapeRules()` method to account for nested
field paths and not just a two-level field path that has the Input or
Output shape name as the first part of the field path and the top-level
member field name as the second part of the field path.
Signed-off-by: Jay Pipes <jaypipes@gmail.com>
By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.1 parent 8287f67 commit d9d3390
1 file changed
+26
-8
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
24 | 25 | | |
| |||
718 | 719 | | |
719 | 720 | | |
720 | 721 | | |
721 | | - | |
722 | | - | |
723 | | - | |
724 | | - | |
725 | | - | |
726 | | - | |
727 | | - | |
728 | | - | |
729 | 722 | | |
730 | 723 | | |
731 | 724 | | |
| |||
739 | 732 | | |
740 | 733 | | |
741 | 734 | | |
| 735 | + | |
| 736 | + | |
| 737 | + | |
| 738 | + | |
| 739 | + | |
| 740 | + | |
| 741 | + | |
| 742 | + | |
| 743 | + | |
| 744 | + | |
| 745 | + | |
| 746 | + | |
| 747 | + | |
| 748 | + | |
| 749 | + | |
| 750 | + | |
| 751 | + | |
| 752 | + | |
| 753 | + | |
| 754 | + | |
| 755 | + | |
| 756 | + | |
| 757 | + | |
| 758 | + | |
| 759 | + | |
742 | 760 | | |
743 | 761 | | |
744 | 762 | | |
| |||
0 commit comments