Murtadha Hubail has submitted this change and it was merged.
Change subject: [ASTERIXDB-2249][BAD] Coordinated Change
......................................................................
[ASTERIXDB-2249][BAD] Coordinated Change
Change-Id: I5287c0b291f2344bd2eeb55e0cb92c387b400ada
---
M
asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java
M
asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateProcedureStatement.java
2 files changed, 4 insertions(+), 3 deletions(-)
Approvals:
Jenkins: Verified
Michael Blow: Looks good to me, approved
diff --git
a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java
b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java
index fcd0c07..bab6ad2 100644
---
a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java
+++
b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java
@@ -187,7 +187,7 @@
metadataProvider.getDefaultDataverse());
tempMdProvider.getConfig().putAll(metadataProvider.getConfig());
- final ResultDelivery resultDelivery =
requestParameters.getResultDelivery();
+ final ResultDelivery resultDelivery =
requestParameters.getResultProperties().getDelivery();
final IHyracksDataset hdc = requestParameters.getHyracksDataset();
final Stats stats = requestParameters.getStats();
if (subscriptionId == null) {
@@ -207,6 +207,7 @@
tempMdProvider
.setResultSerializerFactoryProvider(metadataProvider.getResultSerializerFactoryProvider());
tempMdProvider.setOutputFile(metadataProvider.getOutputFile());
+
tempMdProvider.setMaxResultReads(requestParameters.getResultProperties().getMaxReads());
InsertStatement insert = new InsertStatement(new
Identifier(dataverse),
new Identifier(subscriptionsDatasetName),
subscriptionTuple, varCounter, resultVar, accessor);
diff --git
a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateProcedureStatement.java
b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateProcedureStatement.java
index adfa485..b93f778 100644
---
a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateProcedureStatement.java
+++
b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateProcedureStatement.java
@@ -262,7 +262,7 @@
metadataProvider.getDefaultDataverse());
tempMdProvider.getConfig().putAll(metadataProvider.getConfig());
metadataProvider.setResultSetId(new ResultSetId(resultSetId++));
- final ResultDelivery resultDelivery =
requestParameters.getResultDelivery();
+ final ResultDelivery resultDelivery =
requestParameters.getResultProperties().getDelivery();
final IHyracksDataset hdc = requestParameters.getHyracksDataset();
final Stats stats = requestParameters.getStats();
boolean resultsAsync = resultDelivery == ResultDelivery.ASYNC ||
resultDelivery == ResultDelivery.DEFERRED;
@@ -272,7 +272,7 @@
tempMdProvider.setWriterFactory(metadataProvider.getWriterFactory());
tempMdProvider.setResultSerializerFactoryProvider(metadataProvider.getResultSerializerFactoryProvider());
tempMdProvider.setOutputFile(metadataProvider.getOutputFile());
-
+
tempMdProvider.setMaxResultReads(requestParameters.getResultProperties().getMaxReads());
//Create Procedure Internal Job
Pair<JobSpecification, PrecompiledType> procedureJobSpec =
createProcedureJob(statementExecutor, tempMdProvider, hcc,
hdc, stats);
--
To view, visit https://asterix-gerrit.ics.uci.edu/2293
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5287c0b291f2344bd2eeb55e0cb92c387b400ada
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
