[
https://issues.apache.org/jira/browse/BEAM-8801?focusedWorklogId=360323&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-360323
]
ASF GitHub Bot logged work on BEAM-8801:
----------------------------------------
Author: ASF GitHub Bot
Created on: 16/Dec/19 15:29
Start Date: 16/Dec/19 15:29
Worklog Time Spent: 10m
Work Description: TheNeuralBit commented on pull request #10359:
[BEAM-8801] PubsubMessageToRow should not check useFlatSchema() in pr…
URL: https://github.com/apache/beam/pull/10359#discussion_r358294729
##########
File path:
sdks/java/extensions/sql/src/test/java/org/apache/beam/sdk/extensions/sql/meta/provider/pubsub/PubsubMessageToRowTest.java
##########
@@ -36,23 +36,23 @@
import org.apache.beam.sdk.testing.PAssert;
import org.apache.beam.sdk.testing.TestPipeline;
import org.apache.beam.sdk.transforms.Create;
-import org.apache.beam.sdk.transforms.ParDo;
import org.apache.beam.sdk.values.PCollection;
import org.apache.beam.sdk.values.PCollectionTuple;
import org.apache.beam.sdk.values.Row;
import org.apache.beam.sdk.values.TimestampedValue;
-import org.apache.beam.sdk.values.TupleTagList;
import
org.apache.beam.vendor.calcite.v1_20_0.com.google.common.collect.ImmutableMap;
import
org.apache.beam.vendor.calcite.v1_20_0.com.google.common.collect.ImmutableSet;
import org.joda.time.DateTime;
import org.joda.time.Instant;
+import org.junit.Assert;
import org.junit.Rule;
import org.junit.Test;
/** Unit tests for {@link PubsubMessageToRow}. */
public class PubsubMessageToRowTest implements Serializable {
@Rule public transient TestPipeline pipeline = TestPipeline.create();
+ private static final String DEAD_FILE_QUEUE = "projects/a12345z/topics/test";
Review comment:
nit: can you remove this? I don't think it's used anymore
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 360323)
Time Spent: 2h 40m (was: 2.5h)
> PubsubMessageToRow should not check useFlatSchema() in processElement
> ---------------------------------------------------------------------
>
> Key: BEAM-8801
> URL: https://issues.apache.org/jira/browse/BEAM-8801
> Project: Beam
> Issue Type: Improvement
> Components: dsl-sql
> Reporter: Brian Hulette
> Assignee: Jing Chen
> Priority: Major
> Time Spent: 2h 40m
> Remaining Estimate: 0h
>
> Currently we check useFlatSchema() for every element that's processed.
> Instead, we should check it once at pipeline construction time. See
> [comment|https://github.com/apache/beam/pull/10158#discussion_r348805530].
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
