Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Render all fields in cluster.yaml instead of config files #42

Open
openinx opened this issue Feb 7, 2018 · 0 comments
Open

Render all fields in cluster.yaml instead of config files #42

openinx opened this issue Feb 7, 2018 · 0 comments
Assignees
Milestone

Comments

@openinx
Copy link
Owner

openinx commented Feb 7, 2018

Currently, we only render config files by following:

// If skipHostRender is true , will skip to the HostRender
func (c *Cluster) RenderConfigFiles(job *Job, taskId int, skipHostRender bool) (map[string]string, error) {
	//....
}

Need to render all fields in cluster.yaml, such as java_class , extra_args , because when I want to config the following job, it does not work:

  sqlline_thin:
    super_job: job_common
    main_entry:
      java_class: org.apache.phoenix.queryserver.client.SqllineWrapper
      extra_args:
        -d org.apache.phoenix.queryserver.client.Driver
        -u jdbc:phoenix:thin:url=http://%{queryserver.0.host:queryserver.0.base_port};serialization=PROTOBUF
        -n none
        -p none
        --color=true
        --fastConnect=false
        --verbose=true
        --incremental=false
        --isolation=TRANSACTION_READ_COMMITTED
@openinx openinx self-assigned this Feb 7, 2018
@openinx openinx added this to the release-1.0 milestone Feb 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant