[
https://issues.apache.org/jira/browse/BEAM-7590?focusedWorklogId=265866&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-265866
]
ASF GitHub Bot logged work on BEAM-7590:
----------------------------------------
Author: ASF GitHub Bot
Created on: 24/Jun/19 17:30
Start Date: 24/Jun/19 17:30
Worklog Time Spent: 10m
Work Description: riazela commented on issue #8928: [DO NOT MERGE]
[BEAM-7590] Converting JDBC Pipeline Options Map to PipelineOptions.
URL: https://github.com/apache/beam/pull/8928#issuecomment-505105463
Run Java PreCommit
----------------------------------------------------------------
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:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 265866)
Time Spent: 50m (was: 40m)
> Convert PipelineOptionsMap to PipelineOption
> --------------------------------------------
>
> Key: BEAM-7590
> URL: https://issues.apache.org/jira/browse/BEAM-7590
> Project: Beam
> Issue Type: Improvement
> Components: dsl-sql
> Reporter: Alireza Samadianzakaria
> Assignee: Alireza Samadianzakaria
> Priority: Minor
> Time Spent: 50m
> Remaining Estimate: 0h
>
> Currently, BeamCalciteTable keeps a map version of PipelineOptions and that
> map version is used in JDBCConnection and RelNodes as well. This map is empty
> when the pipeline is constructed from SQLTransform and it will have the
> parameters passed from JDBC Client when the pipeline is started by JDBC path.
> Since for Row-Count estimation we need to use PipelineOptions (or its
> sub-classes) and we cannot convert a map that is created from a
> pipelineOptions Subclasses back to PipelineOptions, it is better to keep
> PipelineOptions object itself.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
