H2Data does not support COLUMNS with ".", "-", or digits

[pinaf]

Fix for issue #7 only dealt with table names. The same Base32 encoding should be done for column names. This is needed for yegor/s3auth#90

[dmarkov]

this task will get someone's attention soon

[dmarkov]

@pinaf many thanks for the bug report, I added 15 mins to your acc, transaction AP-3LH50245RL416922R

[pinaf]

@dmarkov thank you

[dmarkov]

@apocarteres the task is yours

[apocarteres]

@dmarkov i have added test to prove the issue, also added puzzle there. Please find someone to work on that puzzle.

[apocarteres]

@yegor256 what do you think about PR with test and puzzle for this issue? I think it's out of issue's scope to make a decision should H2Data support mentioned functionality or not, moreover to add/remove such functionality.

[pinaf]

@apocarteres We need class H2Data to support such functionality - it currently does not. If you look at issue #7 you'll see the solution was to use Base32 encoding. However #7 only did this for table names. In this issue we need it for column names also.

[apocarteres]

@pinaf i see, let me discuss it with Yegor, please

[apocarteres]

@dmarkov it's needed more time here

[pinaf]

@apocarteres any progress?

[dmarkov]

@dmarkov it's needed more time here

@apocarteres agreed, take your time

[apocarteres]

@pinaf waiting for @yegor256 comments

[yegor256]

@apocarteres sure, reproducing a problem is a good solution. check this article: http://www.yegor256.com/2015/01/15/how-to-cut-corners.html

[apocarteres]

@yegor256 ok thanks
@pinaf well, i have created PR with test to prove that H2Data doesn't support such symbols you mentioned above. Also the puzzle created with ask to implement such functions if they should be implemented. In terms of this task we confirmed (by test) the fact of missing some functions and the next step is guys make new task to solve the puzzle and add requested features then.

[pinaf]

@apocarteres ok, let's get it to pass the code review :)

[apocarteres]

@pinaf test added and merged. please check

[pinaf]

@apocarteres thanks!

[dmarkov]

@apocarteres 30 mins was added to your account, many thanks for your contribution!; +30 to your rating, your total score is +255

[longtimeago]

@yegor256 looks like we need to release new version of jcabi-dynamo to proceed with yegor256/s3auth#90

[pinaf]

@longtimeago also the functionality isn't there yet.

[longtimeago]

@pinaf ahr .... still blocked :(

[0pdd]

@pinaf the puzzle #unknown is still not solved.

[0pdd]

@pinaf the only puzzle #97 is solved here.