Skip to content

Update public to private#52

Closed
597365581 wants to merge 1 commit intounitycatalog:mainfrom
597365581:patch-2
Closed

Update public to private#52
597365581 wants to merge 1 commit intounitycatalog:mainfrom
597365581:patch-2

Conversation

@597365581
Copy link
Contributor

@597365581 597365581 commented Jun 16, 2024

Update public to private

PR Checklist

  • A description of the changes is added to the description of this PR.
  • If there is a related issue, make sure it is linked to this PR.
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added or modified a feature, documentation in docs is updated

Description of changes

Update public to private
@597365581
Copy link
Contributor Author

please help to review

@597365581
Copy link
Contributor Author

@tdas tdas

@dennyglee
Copy link
Contributor

Could you clarify the reason behind switching this from public to private? As well, as reminded to me by @ravivj-db

This is autogenerated java client code using openapi-generator sbt plugin. There are no getters in the class for the variables for which you are trying to change the declaration from public to private. 

TL;DR this change may hamper any code trying to access these variables.

@@ -7,9 +7,9 @@
*/
@javax.annotation.Generated(value = "org.openapitools.codegen.languages.JavaClientCodegen", comments = "Generator version: 7.5.0")
public class ServerVariable {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is generated code. so we do not manually modify this code.

Copy link
Contributor

@tdas tdas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will not accept any changes to generated code. Please close your PR.

@tdas
Copy link
Contributor

tdas commented Jun 20, 2024

Closing this PR. If you have other changes you want to propose in non-generated code, please make another PR.

@tdas tdas closed this Jun 20, 2024
rtyler pushed a commit to rtyler/unitycatalog that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants