Commit 9e3dccf
authored
Fix duplicate binding for spark (prestodb#25729)
## Description
1. Fix duplicate binding for facebook spark
```
1) [Guice/BindingAlreadySet]: ThriftCodecManager was bound multiple times.
Bound at:
1 : PrestoSparkModule.setup(PrestoSparkModule.java:432)
2 : ThriftCodecModule.configure(ThriftCodecModule.java:54)
\_ installed by: PrestoFacebookSparkManifoldMetadataStorageModule -> BlobStoreServiceModule -> SmcClientModule -> ThriftCodecModule
```
## Motivation and Context
<!---Why is this change required? What problem does it solve?-->
<!---If it fixes an open issue, please link to the issue here.-->
## Impact
<!---Describe any public API or user-facing feature change or any
performance impact-->
## Test Plan
1. Internal test passed x12384909565983874
3.
## Contributor checklist
- [ ] Please make sure your submission complies with our [contributing
guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md),
in particular [code
style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style)
and [commit
standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards).
- [ ] PR description addresses the issue accurately and concisely. If
the change is non-trivial, a GitHub Issue is referenced.
- [ ] Documented new properties (with its default value), SQL syntax,
functions, or other functionality.
- [ ] If release notes are required, they follow the [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.
## Release Notes
```
== NO RELEASE NOTE ==
```1 parent e0d3305 commit 9e3dccf
File tree
1 file changed
+2
-2
lines changed- presto-spark-base/src/main/java/com/facebook/presto/spark
1 file changed
+2
-2
lines changedLines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
| 22 | + | |
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| |||
429 | 429 | | |
430 | 430 | | |
431 | 431 | | |
432 | | - | |
| 432 | + | |
433 | 433 | | |
434 | 434 | | |
435 | 435 | | |
| |||
0 commit comments