[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-08 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17317658#comment-17317658
 ] 

Ben Chen commented on KAFKA-12492:
--

Sounds good to me. I will do a follow-up PR.

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
> Fix For: 3.0.0
>
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-08 Thread A. Sophie Blee-Goldman (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17317629#comment-17317629
 ] 

A. Sophie Blee-Goldman commented on KAFKA-12492:


Merged, thanks for the PR! Unfortunately we just barely missed the 2.8 release, 
as John cut the RC earlier today. If you want to see this fix in the 2.8 docs 
then you'll need to submit this exact PR against the kafka-site repo as we 
discussed, but the 2.8 RC is still under vote at the moment so you'd need to 
wait for that to be released at which point the docs in kafka/2.8 will be 
copied over to a new 28 subdirectory in kafka-site. 

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
> Fix For: 3.0.0
>
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-06 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17315284#comment-17315284
 ] 

Ben Chen commented on KAFKA-12492:
--

Do we need to run [local 
tests|https://github.com/apache/kafka/blob/trunk/README.md] and build trigger 
for this task as described here: 
[https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Changes] ?

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-05 Thread A. Sophie Blee-Goldman (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17315186#comment-17315186
 ] 

A. Sophie Blee-Goldman commented on KAFKA-12492:


Yeah, you need to check out the "*Kafka Code Repository*" and not just the 
"*Kafka Website Repository*". The whole setup and process with the docs is kind 
of confusing, I'll see if I can improve the docs/wiki to clear things up a bit. 
Thanks :)

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-05 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17315185#comment-17315185
 ] 

Ben Chen commented on KAFKA-12492:
--

Got it. Thanks. I guess I missed something mentioned here: 
https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Website+Documentation+Changes

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-05 Thread A. Sophie Blee-Goldman (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17315183#comment-17315183
 ] 

A. Sophie Blee-Goldman commented on KAFKA-12492:


Thanks, I'll take a look. But note that you always need to submit a docs PR 
against the kafka-repo, while the kafka-site repo is optional. The reason being 
what I said above, ie during a release the docs from kafka get copied over to 
kafka-site. So if you only fix them in kafka-site but not in kafka, the fix 
will just get wiped out when the next release comes up. 

In general we usually do the kafka PR before the kafka-site PR, for that reason 
among others. But that's not a hard rule or anything -- my point is just that 
you need a PR for the kafka repo as well. 

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-05 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17315173#comment-17315173
 ] 

Ben Chen commented on KAFKA-12492:
--

https://github.com/apache/kafka-site/pull/345/

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-05 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17315172#comment-17315172
 ] 

Ben Chen commented on KAFKA-12492:
--

[~ableegoldman] appreciate for the detailed explanation!

Here is the PR: https://github.com/apache/kafka-site/pull/345/

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-05 Thread A. Sophie Blee-Goldman (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17315140#comment-17315140
 ] 

A. Sophie Blee-Goldman commented on KAFKA-12492:


The kafka-site repo is what the actual, live docs are built from. That's why 
there are separate folders like 27, 26, etc -- these correspond to the docs for 
versions 2.7 and 2.6, and so on. You only need to submit a PR to the kafka-site 
repo if you want your change to show up immediately -- if you don't mind 
waiting for the next release, you can just open a PR to fix the docs in the 
kafka repo directly. Then, these will be copied over to the kafka-site repo and 
made live when the next version is released.

Obviously it would be ideal if we could fix this in all versions, but it's 
probably sufficient to just fix it going forward. The 2.8 release is actually 
going on at the moment, so I would recommend submitting a PR to the kafka repo 
for now. If we can get it merged before 2.8 is released, then we're good -- 
otherwise you can open a followup PR with the same fix in just the 28 
subdirectory of the kafka-site repo.

I'm not sure why you're getting a 403, I was able to setup a local apache 
server to test some docs but that was a while ago. Since it's just a fix of an 
existing formatting error, I wouldn't worry about testing it too much. As long 
as you can figure out why the formatting was messed up to begin with, and feel 
reasonably confident in your fix, then that's good enough. Remember, once the 
fix is in kafka-site it'll be live so you can just see what it looks like then. 
If something is still off, you can always submit a followup PR to fix it right 
away in kafka-site

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-04-01 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17313571#comment-17313571
 ] 

Ben Chen commented on KAFKA-12492:
--

We just need to update the spaces within the section in that html.

But I have few questions.
 # There're few folders in this format: 
kafka-site/\{number}/streams/developer-guide. Are they some history records? I 
should update all of them, or create a new one like 
kafka-site/28/streams/developer-guide with the according changes? (I couldn't 
set kafka-site up. So I cannot try it out.)
 # I tried to setup the local kafka-site by following 
[https://cwiki.apache.org/confluence/display/KAFKA/Setup+Kafka+Website+on+Local+Apache+Server.]
 But I still saw 403 as follows and tried to debug by following 
[https://stackoverflow.com/questions/10873295/error-message-forbidden-you-dont-have-permission-to-access-on-this-server?page=1=votes#tab-top]

             

        Forbidden

           You don't have permission to access this resource.

                     

           Did anyone experience the same issue?

           

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-03-21 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17305797#comment-17305797
 ] 

Ben Chen commented on KAFKA-12492:
--

Thanks so much!

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-03-21 Thread A. Sophie Blee-Goldman (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17305792#comment-17305792
 ] 

A. Sophie Blee-Goldman commented on KAFKA-12492:


Hey [~ben.c], I added you as a contributor so you can self-assign tickets from 
now on. Thanks for picking this up!

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Assignee: Ben Chen
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12492) Formatting of example RocksDBConfigSetter is messed up

2021-03-20 Thread Ben Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17305593#comment-17305593
 ] 

Ben Chen commented on KAFKA-12492:
--

Hi, how can I get assigned for this task? Thanks.

> Formatting of example RocksDBConfigSetter is messed up
> --
>
> Key: KAFKA-12492
> URL: https://issues.apache.org/jira/browse/KAFKA-12492
> Project: Kafka
>  Issue Type: Bug
>  Components: documentation, streams
>Reporter: A. Sophie Blee-Goldman
>Priority: Trivial
>  Labels: docs, newbie
>
> See the example implementation class CustomRocksDBConfig in the docs for the 
> rocksdb.config.setter
> https://kafka.apache.org/documentation/streams/developer-guide/config-streams.html#rocksdb-config-setter



--
This message was sent by Atlassian Jira
(v8.3.4#803005)