Skip to content

Commit

Permalink
fix: java to proto failing
Browse files Browse the repository at this point in the history
- changed java_outer_classname for Permission.proto and Policy.proto
- removed experimental optional from permission proto

Signed-off-by: Theodor Mihalache <[email protected]>
  • Loading branch information
tmihalac committed Aug 6, 2024
1 parent c456222 commit 45009e6
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions protos/feast/core/Permission.proto
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ syntax = "proto3";
package feast.core;

option go_package = "github.com/feast-dev/feast/go/protos/feast/core";
option java_outer_classname = "Permission";
option java_outer_classname = "PermissionProto";
option java_package = "feast.proto.core";

import "feast/core/Policy.proto";
Expand Down Expand Up @@ -43,7 +43,7 @@ message Permission {

bool with_subclasses = 4;

optional string name_pattern = 5;
string name_pattern = 5;

map<string, string> required_tags = 6;

Expand Down
2 changes: 1 addition & 1 deletion protos/feast/core/Policy.proto
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ syntax = "proto3";
package feast.core;

option go_package = "github.com/feast-dev/feast/go/protos/feast/core";
option java_outer_classname = "Policy";
option java_outer_classname = "PolicyProto";
option java_package = "feast.proto.core";

message Policy {
Expand Down
2 changes: 1 addition & 1 deletion sdk/python/feast/permissions/permission.py
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ def to_proto(self) -> PermissionProto:
name=self.name,
types=types,
with_subclasses=self.with_subclasses,
name_pattern=self.name_pattern if self.name_pattern is not None else None,
name_pattern=self.name_pattern if self.name_pattern is not None else "",
actions=actions,
policy=self.policy.to_proto(),
tags=self._tags if self._tags is not None else None,
Expand Down

0 comments on commit 45009e6

Please sign in to comment.