jaeyun0503 commented on code in PR #3827:
URL: https://github.com/apache/texera/pull/3827#discussion_r2412144299
##########
core/workflow-operator/src/main/scala/edu/uci/ics/amber/operator/LogicalOp.scala:
##########
@@ -64,41 +52,11 @@ import edu.uci.ics.amber.operator.regex.RegexOpDesc
import edu.uci.ics.amber.operator.reservoirsampling.ReservoirSamplingOpDesc
import edu.uci.ics.amber.operator.sklearn._
import edu.uci.ics.amber.operator.sleep.SleepOpDesc
-import edu.uci.ics.amber.operator.sklearn.training.{
- SklearnTrainingAdaptiveBoostingOpDesc,
- SklearnTrainingBaggingOpDesc,
- SklearnTrainingBernoulliNaiveBayesOpDesc,
- SklearnTrainingComplementNaiveBayesOpDesc,
- SklearnTrainingDecisionTreeOpDesc,
- SklearnTrainingDummyClassifierOpDesc,
- SklearnTrainingExtraTreeOpDesc,
- SklearnTrainingExtraTreesOpDesc,
- SklearnTrainingGaussianNaiveBayesOpDesc,
- SklearnTrainingGradientBoostingOpDesc,
- SklearnTrainingKNNOpDesc,
- SklearnTrainingLinearRegressionOpDesc,
- SklearnTrainingLinearSVMOpDesc,
- SklearnTrainingLogisticRegressionCVOpDesc,
- SklearnTrainingLogisticRegressionOpDesc,
- SklearnTrainingMultiLayerPerceptronOpDesc,
- SklearnTrainingMultinomialNaiveBayesOpDesc,
- SklearnTrainingNearestCentroidOpDesc,
- SklearnTrainingPassiveAggressiveOpDesc,
- SklearnTrainingPerceptronOpDesc,
- SklearnTrainingProbabilityCalibrationOpDesc,
- SklearnTrainingRandomForestOpDesc,
- SklearnTrainingRidgeCVOpDesc,
- SklearnTrainingRidgeOpDesc,
- SklearnTrainingSDGOpDesc,
- SklearnTrainingSVMOpDesc
-}
+import
edu.uci.ics.amber.operator.sklearn.training.{SklearnTrainingAdaptiveBoostingOpDesc,
SklearnTrainingBaggingOpDesc, SklearnTrainingBernoulliNaiveBayesOpDesc,
SklearnTrainingComplementNaiveBayesOpDesc, SklearnTrainingDecisionTreeOpDesc,
SklearnTrainingDummyClassifierOpDesc, SklearnTrainingExtraTreeOpDesc,
SklearnTrainingExtraTreesOpDesc, SklearnTrainingGaussianNaiveBayesOpDesc,
SklearnTrainingGradientBoostingOpDesc, SklearnTrainingKNNOpDesc,
SklearnTrainingLinearSVMOpDesc, SklearnTrainingMultiLayerPerceptronOpDesc,
SklearnTrainingMultinomialNaiveBayesOpDesc,
SklearnTrainingNearestCentroidOpDesc, SklearnTrainingPassiveAggressiveOpDesc,
SklearnTrainingPerceptronOpDesc, SklearnTrainingProbabilityCalibrationOpDesc,
SklearnTrainingRandomForestOpDesc, SklearnTrainingRidgeCVOpDesc,
SklearnTrainingRidgeOpDesc, SklearnTrainingSDGOpDesc, SklearnTrainingSVMOpDesc}
Review Comment:
Ditto
##########
core/workflow-operator/src/main/scala/edu/uci/ics/amber/operator/LogicalOp.scala:
##########
@@ -39,22 +35,14 @@ import edu.uci.ics.amber.operator.distinct.DistinctOpDesc
import edu.uci.ics.amber.operator.dummy.DummyOpDesc
import edu.uci.ics.amber.operator.filter.SpecializedFilterOpDesc
import edu.uci.ics.amber.operator.hashJoin.HashJoinOpDesc
-import edu.uci.ics.amber.operator.huggingFace.{
- HuggingFaceIrisLogisticRegressionOpDesc,
- HuggingFaceSentimentAnalysisOpDesc,
- HuggingFaceSpamSMSDetectionOpDesc,
- HuggingFaceTextSummarizationOpDesc
-}
+import
edu.uci.ics.amber.operator.huggingFace.{HuggingFaceIrisLogisticRegressionOpDesc,
HuggingFaceSentimentAnalysisOpDesc, HuggingFaceSpamSMSDetectionOpDesc,
HuggingFaceTextSummarizationOpDesc}
Review Comment:
Ditto
##########
core/workflow-operator/src/main/scala/edu/uci/ics/amber/operator/LogicalOp.scala:
##########
@@ -64,41 +52,11 @@ import edu.uci.ics.amber.operator.regex.RegexOpDesc
import edu.uci.ics.amber.operator.reservoirsampling.ReservoirSamplingOpDesc
import edu.uci.ics.amber.operator.sklearn._
import edu.uci.ics.amber.operator.sleep.SleepOpDesc
-import edu.uci.ics.amber.operator.sklearn.training.{
- SklearnTrainingAdaptiveBoostingOpDesc,
- SklearnTrainingBaggingOpDesc,
- SklearnTrainingBernoulliNaiveBayesOpDesc,
- SklearnTrainingComplementNaiveBayesOpDesc,
- SklearnTrainingDecisionTreeOpDesc,
- SklearnTrainingDummyClassifierOpDesc,
- SklearnTrainingExtraTreeOpDesc,
- SklearnTrainingExtraTreesOpDesc,
- SklearnTrainingGaussianNaiveBayesOpDesc,
- SklearnTrainingGradientBoostingOpDesc,
- SklearnTrainingKNNOpDesc,
- SklearnTrainingLinearRegressionOpDesc,
- SklearnTrainingLinearSVMOpDesc,
- SklearnTrainingLogisticRegressionCVOpDesc,
- SklearnTrainingLogisticRegressionOpDesc,
- SklearnTrainingMultiLayerPerceptronOpDesc,
- SklearnTrainingMultinomialNaiveBayesOpDesc,
- SklearnTrainingNearestCentroidOpDesc,
- SklearnTrainingPassiveAggressiveOpDesc,
- SklearnTrainingPerceptronOpDesc,
- SklearnTrainingProbabilityCalibrationOpDesc,
- SklearnTrainingRandomForestOpDesc,
- SklearnTrainingRidgeCVOpDesc,
- SklearnTrainingRidgeOpDesc,
- SklearnTrainingSDGOpDesc,
- SklearnTrainingSVMOpDesc
-}
+import
edu.uci.ics.amber.operator.sklearn.training.{SklearnTrainingAdaptiveBoostingOpDesc,
SklearnTrainingBaggingOpDesc, SklearnTrainingBernoulliNaiveBayesOpDesc,
SklearnTrainingComplementNaiveBayesOpDesc, SklearnTrainingDecisionTreeOpDesc,
SklearnTrainingDummyClassifierOpDesc, SklearnTrainingExtraTreeOpDesc,
SklearnTrainingExtraTreesOpDesc, SklearnTrainingGaussianNaiveBayesOpDesc,
SklearnTrainingGradientBoostingOpDesc, SklearnTrainingKNNOpDesc,
SklearnTrainingLinearSVMOpDesc, SklearnTrainingMultiLayerPerceptronOpDesc,
SklearnTrainingMultinomialNaiveBayesOpDesc,
SklearnTrainingNearestCentroidOpDesc, SklearnTrainingPassiveAggressiveOpDesc,
SklearnTrainingPerceptronOpDesc, SklearnTrainingProbabilityCalibrationOpDesc,
SklearnTrainingRandomForestOpDesc, SklearnTrainingRidgeCVOpDesc,
SklearnTrainingRidgeOpDesc, SklearnTrainingSDGOpDesc, SklearnTrainingSVMOpDesc}
import edu.uci.ics.amber.operator.sort.SortOpDesc
import edu.uci.ics.amber.operator.sortPartitions.SortPartitionsOpDesc
import edu.uci.ics.amber.operator.source.apis.reddit.RedditSearchSourceOpDesc
-import edu.uci.ics.amber.operator.source.apis.twitter.v2.{
- TwitterFullArchiveSearchSourceOpDesc,
- TwitterSearchSourceOpDesc
-}
+import
edu.uci.ics.amber.operator.source.apis.twitter.v2.{TwitterFullArchiveSearchSourceOpDesc,
TwitterSearchSourceOpDesc}
Review Comment:
Ditto
##########
core/workflow-operator/src/main/scala/edu/uci/ics/amber/operator/LogicalOp.scala:
##########
@@ -24,11 +24,7 @@ import com.fasterxml.jackson.annotation._
import com.kjetland.jackson.jsonSchema.annotations.JsonSchemaTitle
import edu.uci.ics.amber.core.executor.OperatorExecutor
import edu.uci.ics.amber.core.tuple.Schema
-import edu.uci.ics.amber.core.virtualidentity.{
- ExecutionIdentity,
- OperatorIdentity,
- WorkflowIdentity
-}
+import edu.uci.ics.amber.core.virtualidentity.{ExecutionIdentity,
OperatorIdentity, WorkflowIdentity}
Review Comment:
Please avoid making changes to other parts of the code that is unrelated to
your PR.
##########
core/workflow-operator/src/main/scala/edu/uci/ics/amber/operator/LogicalOp.scala:
##########
@@ -39,22 +35,14 @@ import edu.uci.ics.amber.operator.distinct.DistinctOpDesc
import edu.uci.ics.amber.operator.dummy.DummyOpDesc
import edu.uci.ics.amber.operator.filter.SpecializedFilterOpDesc
import edu.uci.ics.amber.operator.hashJoin.HashJoinOpDesc
-import edu.uci.ics.amber.operator.huggingFace.{
- HuggingFaceIrisLogisticRegressionOpDesc,
- HuggingFaceSentimentAnalysisOpDesc,
- HuggingFaceSpamSMSDetectionOpDesc,
- HuggingFaceTextSummarizationOpDesc
-}
+import
edu.uci.ics.amber.operator.huggingFace.{HuggingFaceIrisLogisticRegressionOpDesc,
HuggingFaceSentimentAnalysisOpDesc, HuggingFaceSpamSMSDetectionOpDesc,
HuggingFaceTextSummarizationOpDesc}
import edu.uci.ics.amber.operator.ifStatement.IfOpDesc
import edu.uci.ics.amber.operator.intersect.IntersectOpDesc
import edu.uci.ics.amber.operator.intervalJoin.IntervalJoinOpDesc
import edu.uci.ics.amber.operator.keywordSearch.KeywordSearchOpDesc
import edu.uci.ics.amber.operator.limit.LimitOpDesc
import
edu.uci.ics.amber.operator.machineLearning.Scorer.MachineLearningScorerOpDesc
-import edu.uci.ics.amber.operator.machineLearning.sklearnAdvanced.KNNTrainer.{
- SklearnAdvancedKNNClassifierTrainerOpDesc,
- SklearnAdvancedKNNRegressorTrainerOpDesc
-}
+import
edu.uci.ics.amber.operator.machineLearning.sklearnAdvanced.KNNTrainer.{SklearnAdvancedKNNClassifierTrainerOpDesc,
SklearnAdvancedKNNRegressorTrainerOpDesc}
Review Comment:
Ditto
--
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]
