damondouglas commented on a change in pull request #15660:
URL: https://github.com/apache/beam/pull/15660#discussion_r724259256
##########
File path: playground/backend/internal/executors/executor.go
##########
@@ -13,19 +13,85 @@
// See the License for the specific language governing permissions and
// limitations under the License.
-// Interface for all executors (Java/Python/Go/SCIO)
+// Package executors
package executors
-type executor interface {
- // Validate validates executable file.
- // Return result of validation (true/false) and error if it occurs
- Validate(filePath string) (bool, error)
+import (
+ pb "beam.apache.org/playground/backend/internal/api"
+ "beam.apache.org/playground/backend/internal/fs_tool"
+ "fmt"
+ "os/exec"
+)
- // Compile compiles executable file.
- // Return error if it occurs
- Compile(filePath string) error
+type validatorWithArgs struct {
+ validator func(filePath string, args ...interface{}) error
Review comment:
This is not blocking of the PR but I think this might be a good
candidate for an interface in the future.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
